On Fri, Apr 22, 2011 at 8:52 AM, Dan Nicholson <dbn.li...@gmail.com> wrote: > On Thu, Apr 21, 2011 at 4:39 AM, Marek Olšák <mar...@gmail.com> wrote: >> OpenSUSE is (was?) shipping r300g with LLVM disabled. Can you believe it? >> Poor users with SWTCL chipsets... and bad reputation for us too. >> --- >> configure.ac | 4 ++-- >> 1 files changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/configure.ac b/configure.ac >> index 8989c2b..8e9f73f 100644 >> --- a/configure.ac >> +++ b/configure.ac >> @@ -1687,10 +1687,10 @@ dnl Gallium LLVM >> dnl >> AC_ARG_ENABLE([gallium-llvm], >> [AS_HELP_STRING([--enable-gallium-llvm], >> - [build gallium LLVM support @<:@default=disabled@:>@])], >> + [build gallium LLVM support @<:@default=enabled@:>@])], >> [enable_gallium_llvm="$enableval"], >> [enable_gallium_llvm=auto]) >> -if test "x$enable_gallium_llvm" = xyes; then >> +if test "x$enable_gallium_llvm" != xno; then >> if test "x$LLVM_CONFIG" != xno; then >> LLVM_VERSION=`$LLVM_CONFIG --version` >> LLVM_CFLAGS=`$LLVM_CONFIG --cppflags`
Hit send a little too fast. This looks right, although the default is really "auto" and will silently continue if the user doesn't have LLVM. That's a little different than "enabled". It would be nice for the user if configure errors when LLVM_CONFIG=no and enable_gallium_llvm=yes. -- Dan _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
