Re: [PATCH] Fix VIS3 assembler check and conditionalize testsuite on VIS3 support.

[David Miller]

From: Eric Botcazou <ebotca...@adacore.com>
Date: Tue, 11 Oct 2011 21:57:18 +0200

>> I would expect that to spit out a warning.  Do I need to explicitly
>> add "-Wall", "-Wno-implicit" or similar?  Similar tests in i386.exp don't
>> seem to need this and that was what I used as my template.
> 
> -Wall does yield a warning:
> 
> vis.c: In function '_vis3_fpadd64':
> vis.c:4:3: warning: implicit declaration of function '__builtin_vis_fpadd64' 
> [-Wimplicit-function-declaration]

Cool, Eric could you quickly test the following?  This still leaves
the i386.exp case issue open, it stands to reason that something like
-Wall is needed for those tests too.

diff --git a/gcc/testsuite/gcc.target/sparc/sparc.exp 
b/gcc/testsuite/gcc.target/sparc/sparc.exp
index 51c9c16..7d30dcc 100644
--- a/gcc/testsuite/gcc.target/sparc/sparc.exp
+++ b/gcc/testsuite/gcc.target/sparc/sparc.exp
@@ -32,7 +32,7 @@ proc check_effective_target_vis3 { } {
        {
            return __builtin_vis_fpadd64 (__X, __Y);
        }
-    } "-mcpu=niagara3 -mvis" ]
+    } "-mcpu=niagara3 -mvis -Wall" ]
 }
 
 # If a testcase doesn't have special options, use these.