diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog index 6886b965b27..b6d945ff53d 100644 --- a/gcc/cp/ChangeLog +++ b/gcc/cp/ChangeLog @@ -1,3 +1,10 @@ +2014-05-20 Paolo Carlini + + PR c++/60373 + * decl.c (duplicate_decls): Replace pair of warning_at with + warning_at + inform. + (maybe_commonize_var): Likewise. + 2014-05-20 Richard Sandiford PR bootstrap/61210 diff --git a/gcc/cp/decl.c b/gcc/cp/decl.c index 81f127315ec..ab91928cf42 100644 --- a/gcc/cp/decl.c +++ b/gcc/cp/decl.c @@ -2306,10 +2306,11 @@ duplicate_decls (tree newdecl, tree olddecl, bool newdecl_is_friend) && DECL_VISIBILITY_SPECIFIED (newdecl) && DECL_VISIBILITY (newdecl) != DECL_VISIBILITY (olddecl)) { - warning_at (input_location, OPT_Wattributes, - "%q+D: visibility attribute ignored because it", newdecl); - warning_at (DECL_SOURCE_LOCATION (olddecl), OPT_Wattributes, - "conflicts with previous declaration here"); + if (warning_at (DECL_SOURCE_LOCATION (newdecl), OPT_Wattributes, + "%qD: visibility attribute ignored because it " + "conflicts with previous declaration", newdecl)) + inform (DECL_SOURCE_LOCATION (olddecl), + "previous declaration of %qD", olddecl); } /* Choose the declaration which specified visibility. */ if (DECL_VISIBILITY_SPECIFIED (olddecl)) @@ -5026,13 +5027,12 @@ maybe_commonize_var (tree decl) be merged. */ TREE_PUBLIC (decl) = 0; DECL_COMMON (decl) = 0; - warning_at (input_location, 0, - "sorry: semantics of inline function static " - "data %q+#D are wrong (you%'ll wind up " - "with multiple copies)", decl); - warning_at (DECL_SOURCE_LOCATION (decl), 0, - " you can work around this by removing " - "the initializer"); + if (warning_at (DECL_SOURCE_LOCATION (decl), 0, + "sorry: semantics of inline function static " + "data %q#D are wrong (you%'ll wind up " + "with multiple copies)", decl)) + inform (DECL_SOURCE_LOCATION (decl), + "you can work around this by removing the initializer"); } } } diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 1440e049912..be9a7def5c4 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,10 @@ +2014-05-20 Paolo Carlini + + PR c++/60373 + * g++.dg/cpp0x/Wattributes1.C: New. + * g++.dg/ext/visibility/redecl1.C: Adjust. + * g++.dg/ext/visibility/visibility-7.C: Likewise. + 2014-05-20 Bill Schmidt * gcc.target/powerpc/vsxcopy.c: New test. diff --git a/gcc/testsuite/g++.dg/cpp0x/Wattributes1.C b/gcc/testsuite/g++.dg/cpp0x/Wattributes1.C new file mode 100644 index 00000000000..4d3c29ce3f1 --- /dev/null +++ b/gcc/testsuite/g++.dg/cpp0x/Wattributes1.C @@ -0,0 +1,8 @@ +// PR c++/60373 +// { dg-do compile { target c++11 } } +// { dg-require-visibility "" } + +#include +__attribute__((visibility("hidden")))void*operator new(std::size_t); // { dg-warning "visibility attribute ignored" } + +// { dg-message "previous declaration" "" { target *-*-* } 128 } diff --git a/gcc/testsuite/g++.dg/ext/visibility/redecl1.C b/gcc/testsuite/g++.dg/ext/visibility/redecl1.C index b53335ef583..a85e6c38e95 100644 --- a/gcc/testsuite/g++.dg/ext/visibility/redecl1.C +++ b/gcc/testsuite/g++.dg/ext/visibility/redecl1.C @@ -3,5 +3,5 @@ struct __attribute((visibility("hidden"))) B; struct __attribute((visibility("default"))) B; // { dg-error "visibility" } -__attribute ((visibility ("hidden"))) void f(); // { dg-warning "previous" } +__attribute ((visibility ("hidden"))) void f(); // { dg-message "previous" } __attribute ((visibility ("default"))) void f(); // { dg-warning "visibility" } diff --git a/gcc/testsuite/g++.dg/ext/visibility/visibility-7.C b/gcc/testsuite/g++.dg/ext/visibility/visibility-7.C index ae4589397b6..0a6e4ba637b 100644 --- a/gcc/testsuite/g++.dg/ext/visibility/visibility-7.C +++ b/gcc/testsuite/g++.dg/ext/visibility/visibility-7.C @@ -4,7 +4,7 @@ extern int __attribute__((visibility ("hidden"))) -xyzzy; /* { dg-warning "previous declaration here" "" } */ +xyzzy; /* { dg-message "previous declaration" "" } */ int __attribute__((visibility ("protected")))