On 19 December 2016 at 08:52, Tomasz Figa <tf...@chromium.org> wrote: > Hi Tobias, > > On Sat, Dec 17, 2016 at 2:15 AM, Tobias Droste <tdro...@gmx.de> wrote: >> Hi Tomasz, >> >> does this actually fix anything? >> >> Because right now llvm-config.h does not include anything and I doubt it will >> in the future, as it's just a collection of defines. >> The path to the header file itself is given by llvm-config >> ($LLVM_INCLUDEDIR). >> >> Did you just happen to see this or do you get an error without this patch? > > We happen to have the setup exactly as described in the patch > description, i.e. LLVM in a non-standard directory and with > llvm-config.h that includes another header. Without the patch > ./configure fails because of LLVM version detection errors. > I believe you're spot on - be that due to missing include [directives] or conditional header inclusion one would need to have the respective CFLAGS. Thus the patch is Reviewed-by: Emil Velikov <emil.veli...@collabora.com>
At the same time - I seems to be lucky enough to have the headers sane [neither includes nor defines are required] on my Arch setup. Wondering if it's not something specific to the Android way of building LLVM - IIRC the Android-x86 guys were explaining some interesting things. ... but that for another day. I'll commit your patch once I double-check Tobias' fixes. Thanks Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev