diff options
author | Jakub Jelinek <jakub@redhat.com> | 2019-02-28 08:17:49 +0100 |
---|---|---|
committer | Jakub Jelinek <jakub@gcc.gnu.org> | 2019-02-28 08:17:49 +0100 |
commit | 66dcb747e8ba1f0ed6e0d49ed3d842b3e091609d (patch) | |
tree | f9e4f0bf17c9733435280d2b609d2b9ca4dca8f9 /gcc/c/c-typeck.c | |
parent | c2df3c99d5cafbb9774fc6e944fbd64aa17c03bb (diff) |
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
Diffstat (limited to 'gcc/c/c-typeck.c')
-rw-r--r-- | gcc/c/c-typeck.c | 18 |
1 files changed, 8 insertions, 10 deletions
diff --git a/gcc/c/c-typeck.c b/gcc/c/c-typeck.c index 48112006f04..2de4d0f85fd 100644 --- a/gcc/c/c-typeck.c +++ b/gcc/c/c-typeck.c @@ -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; |