[llvm] r191289 - Remove error output from configure if CFLAGS is set (r174313).

Patrik Hagglund patrik.h.hagglund at ericsson.com
Tue Sep 24 04:38:46 PDT 2013


Author: patha
Date: Tue Sep 24 06:38:45 2013
New Revision: 191289

URL: http://llvm.org/viewvc/llvm-project?rev=191289&view=rev
Log:
Remove error output from configure if CFLAGS is set (r174313).

This fixes PR16724.

Modified:
    llvm/trunk/autoconf/configure.ac
    llvm/trunk/configure

Modified: llvm/trunk/autoconf/configure.ac
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/autoconf/configure.ac?rev=191289&r1=191288&r2=191289&view=diff
==============================================================================
--- llvm/trunk/autoconf/configure.ac (original)
+++ llvm/trunk/autoconf/configure.ac Tue Sep 24 06:38:45 2013
@@ -61,8 +61,8 @@ fi
 
 dnl Default to empty (i.e. assigning the null string to) CFLAGS and CXXFLAGS,
 dnl instead of the autoconf default (for example, '-g -O2' for CC=gcc).
-${CFLAGS=}
-${CXXFLAGS=}
+: ${CFLAGS=}
+: ${CXXFLAGS=}
 
 dnl We need to check for the compiler up here to avoid anything else
 dnl starting with a different one.

Modified: llvm/trunk/configure
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/configure?rev=191289&r1=191288&r2=191289&view=diff
==============================================================================
--- llvm/trunk/configure (original)
+++ llvm/trunk/configure Tue Sep 24 06:38:45 2013
@@ -1992,8 +1992,8 @@ echo "$as_me: error: Already configured
   fi
 fi
 
-${CFLAGS=}
-${CXXFLAGS=}
+: ${CFLAGS=}
+: ${CXXFLAGS=}
 
 ac_ext=c
 ac_cpp='$CPP $CPPFLAGS'





More information about the llvm-commits mailing list