re PR c/89525 (inform messages from -Wbuiltin-declaration-mismatch even with -w)

PR c/89525
	* c-typeck.c (convert_arguments): Call inform_declaration only if
	the previous warning_at call returned true.

	* gcc.dg/pr89525.c: New test.

From-SVN: r269274
This commit is contained in:
Jakub Jelinek 2019-02-28 08:17:49 +01:00 committed by Jakub Jelinek
parent c2df3c99d5
commit 66dcb747e8
4 changed files with 28 additions and 10 deletions

View File

@ -1,3 +1,9 @@
2019-02-28 Jakub Jelinek <jakub@redhat.com>
PR c/89525
* c-typeck.c (convert_arguments): Call inform_declaration only if
the previous warning_at call returned true.
2019-02-22 Thomas Schwinge <thomas@codesourcery.com>
* c-parser.c (c_parser_oacc_shape_clause): Add loc formal

View File

@ -3509,12 +3509,10 @@ convert_arguments (location_t loc, vec<location_t> arg_loc, tree typelist,
if (builtin_type == void_type_node)
{
warning_at (loc, OPT_Wbuiltin_declaration_mismatch,
"too many arguments to built-in function %qE "
"expecting %d",
function, parmnum);
inform_declaration (fundecl);
if (warning_at (loc, OPT_Wbuiltin_declaration_mismatch,
"too many arguments to built-in function %qE "
"expecting %d", function, parmnum))
inform_declaration (fundecl);
builtin_typetail = NULL_TREE;
}
@ -3651,10 +3649,10 @@ convert_arguments (location_t loc, vec<location_t> arg_loc, tree typelist,
for (tree t = builtin_typetail; t; t = TREE_CHAIN (t))
++nargs;
warning_at (loc, OPT_Wbuiltin_declaration_mismatch,
"too few arguments to built-in function %qE expecting %u",
function, nargs - 1);
inform_declaration (fundecl);
if (warning_at (loc, OPT_Wbuiltin_declaration_mismatch,
"too few arguments to built-in function %qE "
"expecting %u", function, nargs - 1))
inform_declaration (fundecl);
}
return error_args ? -1 : (int) parmnum;

View File

@ -1,5 +1,8 @@
2019-02-28 Jakub Jelinek <jakub@redhat.com>
PR c/89525
* gcc.dg/pr89525.c: New test.
PR c/89520
* gcc.dg/pr89520-1.c: New test.
* gcc.dg/pr89520-2.c: New test.

View File

@ -0,0 +1,11 @@
/* PR c/89525 */
/* { dg-do compile } */
/* { dg-options "-w" } */
double sqrt (); /* { dg-bogus "declared here" } */
void
foo (void)
{
sqrt ();
}