[Mesa-dev] [PATCH] configure.ac: Respect LLVM_CFLAGS in LLVM version detection

Tomasz Figa tfiga at chromium.org
Mon Dec 19 08:52:38 UTC 2016


Hi Tobias,

On Sat, Dec 17, 2016 at 2:15 AM, Tobias Droste <tdroste at 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.

Best regards,
Tomasz


More information about the mesa-dev mailing list