<div dir="auto"><span style="font-family:sans-serif;font-size:11.008px">Reviewed-by: Marek Olšák <</span><a href="mailto:marek.olsak@amd.com" style="text-decoration:none;color:rgb(66,133,244);font-family:sans-serif;font-size:11.008px">marek.olsak@amd.com</a><span style="font-family:sans-serif;font-size:11.008px">></span></div><div class="gmail_extra"><br><div class="gmail_quote">On Jan 26, 2017 12:13 PM, "Andreas Boll" <<a href="mailto:andreas.boll.dev@gmail.com">andreas.boll.dev@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">b3119a3 introduced a strict LLVM requirement for r300 on all<br>
architectures and thus configure fails on architectures where LLVM is<br>
not available or buggy.<br>
<br>
r300 doesn't strictly require LLVM, but for performance reasons we<br>
highly recommend LLVM usage. So require it at least on x86 and x86_64<br>
architectures as we have done before b3119a3.<br>
<br>
Fixes: b3119a3 ("<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a>: Check gallium LLVM version in gallium_require_llvm")<br>
Cc: 17.0 <<a href="mailto:mesa-stable@lists.freedesktop.org">mesa-stable@lists.<wbr>freedesktop.org</a>><br>
Signed-off-by: Andreas Boll <<a href="mailto:andreas.boll.dev@gmail.com">andreas.boll.dev@gmail.com</a>><br>
---<br>
<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> | 15 ++++++++++++++-<br>
1 file changed, 14 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> b/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
index 64ace9d..b35adc8 100644<br>
--- a/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
+++ b/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
@@ -2213,6 +2213,19 @@ gallium_require_llvm() {<br>
}<br>
<br>
dnl<br>
+dnl r300 doesn't strictly require LLVM, but for performance reasons we<br>
+dnl highly recommend LLVM usage. So require it at least on x86 and x86_64<br>
+dnl architectures.<br>
+dnl<br>
+r300_require_llvm() {<br>
+ case "$host" in *gnux32) return;; esac<br>
+ case "$host_cpu" in<br>
+ i*86|x86_64|amd64) gallium_require_llvm $1<br>
+ ;;<br>
+ esac<br>
+}<br>
+<br>
+dnl<br>
dnl DRM is needed by X, Wayland, and offscreen rendering.<br>
dnl Surfaceless is an alternative for the last one.<br>
dnl<br>
@@ -2298,7 +2311,7 @@ if test -n "$with_gallium_drivers"; then<br>
HAVE_GALLIUM_R300=yes<br>
PKG_CHECK_MODULES([RADEON], [libdrm_radeon >= $LIBDRM_RADEON_REQUIRED])<br>
require_libdrm "r300"<br>
- gallium_require_llvm "r300"<br>
+ r300_require_llvm "r300"<br>
;;<br>
xr600)<br>
HAVE_GALLIUM_R600=yes<br>
--<br>
2.1.4<br>
<br>
______________________________<wbr>_________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
</blockquote></div></div>