<div dir="auto">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.</div><div class="gmail_extra"> <div class="gmail_quote">On Jan 28, 2017 8:57 AM, "Tobias Droste" <<a href="mailto:tdroste@gmx.de">tdroste@gmx.de</a>> wrote: <blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This is a reworked series of the previous LLVM related changes to <a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> 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): <a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a>: Rename MESA_LLVM to FOUND_LLVM <a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a>: Only set LLVM_LIBS if LLVM is used <a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a>: Separate HAVE_LLVM defines for gallium and radv <a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a>: Don't check LLVM version in gallium_require_llvm <a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> | 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 <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a> <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a> </blockquote></div> </div>