[llvm-commits] [LNT] r155854 - /lnt/trunk/lnt/tests/compile.py

[Daniel Dunbar]

Author: ddunbar
Date: Mon Apr 30 18:05:46 2012
New Revision: 155854

URL: http://llvm.org/viewvc/llvm-project?rev=155854&view=rev
Log:
lnt.tests.compile: Run tests with -w, we shouldn't penalize newer compilers for
adding betters warnings.

Modified:
    lnt/trunk/lnt/tests/compile.py

Modified: lnt/trunk/lnt/tests/compile.py
URL: http://llvm.org/viewvc/llvm-project/lnt/trunk/lnt/tests/compile.py?rev=155854&r1=155853&r2=155854&view=diff
==============================================================================
--- lnt/trunk/lnt/tests/compile.py (original)
+++ lnt/trunk/lnt/tests/compile.py Mon Apr 30 18:05:46 2012
@@ -126,8 +126,8 @@
     cmd.append(input)
     cmd.extend(flags)
 
-    # FIXME: Probably a bad idea, but its not worth worry about test failures
-    # for compiler versions which emit warnings on the test inputs for now.
+    # Inhibit all warnings, we don't want to count the time to generate them
+    # against newer compilers which have added (presumably good) warnings.
     cmd.append('-w')
 
     # Do a memory profiling run, if requested.
@@ -349,6 +349,10 @@
         # (we don't want to obscure what we are trying to time).
         cmd.append('RUN_CLANG_STATIC_ANALYZER=NO')
 
+        # Inhibit all warnings, we don't want to count the time to generate them
+        # against newer compilers which have added (presumably good) warnings.
+        cmd.append('GCC_WARN_INHIBIT_ALL_WARNINGS=YES')
+
         # Add additional arguments to force the build scenario we want.
         cmd.extend(('-jobs', str(num_jobs)))
     else: