[Mesa-dev] [PATCH 0/4] configure.ac: LLVM again!

Tobias Droste tdroste at gmx.de
Sat Jan 28 20:25:00 UTC 2017


This is what was allowed from the beginning, this just unbreaks the build.

I had a different solution for this, but this was shut down by Jose.

See:

https://lists.freedesktop.org/archives/mesa-dev/2017-January/141263.html
and
https://bugs.freedesktop.org/show_bug.cgi?id=99010

Am Samstag, 28. Januar 2017, 09:31:28 CET schrieb Ilia Mirkin:
> Can you explain why it's a desirable goal to be able to build radv without
> --enable-gallium-llvm? Perhaps it's obvious, but I'm not seeing it.
> 
> On Jan 28, 2017 8:57 AM, "Tobias Droste" <tdroste at gmx.de> wrote:
> 
> This is a reworked series of the previous LLVM related changes to
> configure.ac that were reverted due to breaking scons.
> 
> This takes a different approach to the previous series and adds an
> extra define for LLVM version checks if RADV is build.
> 
> This allows to build RADV with "--disable-gallium-llvm".
> 
> Patch 1, 2 and 3 are the same as in the previous series, just rebased.
> The new stuff is in patch 3.
> 
> Tobias Droste (4):
>   configure.ac: Rename MESA_LLVM to FOUND_LLVM
>   configure.ac: Only set LLVM_LIBS if LLVM is used
>   configure.ac: Separate HAVE_LLVM defines for gallium and radv
>   configure.ac: Don't check LLVM version in gallium_require_llvm
> 
>  configure.ac                      | 49 ++++++++++++++++++++++--------
> ---------
>  src/amd/common/ac_llvm_helper.cpp |  8 ++++++-
>  src/amd/common/ac_llvm_util.c     | 17 ++++++++++----
>  3 files changed, 47 insertions(+), 27 deletions(-)
> 
> --
> 2.11.0
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list