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.
