<div class="gmail_quote">On Fri, Apr 22, 2011 at 1:29 PM, Jose Fonseca <span dir="ltr"><<a href="mailto:jfonseca@vmware.com">jfonseca@vmware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div><div></div>The Mesa state tracker uses SWTNL for GL selection/feedback regardless of the driver. Some SPECviewperf viewsets and CAD apps use it. So using LLVM speeds up selection/feedback for all gallium drivers.<br>
</div>
<br>
We have only tested LLVM with x86/x86_64. So indeed, using it/requiring it on other platforms is not advisable.<br></blockquote></div><br>I take the first patch back. Here's an updated r300g patch that requires LLVM on x86 and x86_64 only:<br>
<br><br> <a href="http://configure.ac">configure.ac</a>: require LLVM to build r300g on x86 and x86_64<br><br>diff --git a/<a href="http://configure.ac">configure.ac</a> b/<a href="http://configure.ac">configure.ac</a><br>
index d8c50ce..1012ca5 100644<br>--- a/<a href="http://configure.ac">configure.ac</a><br>+++ b/<a href="http://configure.ac">configure.ac</a><br>@@ -1780,9 +1780,16 @@ dnl Gallium Radeon r300g configuration<br> dnl<br> AC_ARG_ENABLE([gallium-r300],<br>
[AS_HELP_STRING([--enable-gallium-r300],<br>- [build gallium r300 @<:@default=DRI-only@:>@])],<br>+ [build gallium r300 @<:@default=build DRI driver only@:>@])],<br> [enable_gallium_r300="$enableval"],<br>
[enable_gallium_r300=auto])<br>+if test "x$enable_gallium_r300" != xno; then<br>+ if test "x$MESA_LLVM" = x0; then<br>+ case "$host_cpu" in<br>+ i*86|x86_64) AC_MSG_ERROR([LLVM is required to build Gallium R300 on x86 and x86_64]);;<br>
+ esac<br>+ fi<br>+fi<br> if test "x$enable_gallium_r300" = xauto; then<br> GALLIUM_DRIVERS_DIRS="$GALLIUM_DRIVERS_DIRS r300"<br> gallium_check_st "radeon/drm" "dri-r300"<br>
<br><br>Marek<br>