From 657e4e47344408f7ca6b3c05bd86337dd424a3cf Mon Sep 17 00:00:00 2001 From: Jakub Jelinek Date: Wed, 2 Dec 2015 08:33:06 +0100 Subject: =?UTF-8?q?re=20PR=20c/68533=20(bogus=20location=20for=20"warning:?= =?UTF-8?q?=20=E2=80=98struct=20s3=E2=80=99=20declared=20inside=20paramete?= =?UTF-8?q?r=20list=20will=20not=20be=20visible=20outside=20of=20this=20de?= =?UTF-8?q?finition=20or=20declaration")?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit PR c/68533 * c-decl.c (get_parm_info): Use b->locus instead of input_location for diagnostics. * gcc.dg/pr68533.c: New test. From-SVN: r231147 --- gcc/c/ChangeLog | 6 ++++++ gcc/c/c-decl.c | 13 +++++++------ 2 files changed, 13 insertions(+), 6 deletions(-) (limited to 'gcc/c') diff --git a/gcc/c/ChangeLog b/gcc/c/ChangeLog index 43d1579..acb8ee4 100644 --- a/gcc/c/ChangeLog +++ b/gcc/c/ChangeLog @@ -1,3 +1,9 @@ +2015-12-02 Jakub Jelinek + + PR c/68533 + * c-decl.c (get_parm_info): Use b->locus instead of input_location + for diagnostics. + 2015-12-01 Julian Brown Cesar Philippidis James Norris diff --git a/gcc/c/c-decl.c b/gcc/c/c-decl.c index 31de0a5..efb0a52 100644 --- a/gcc/c/c-decl.c +++ b/gcc/c/c-decl.c @@ -6913,11 +6913,11 @@ get_parm_info (bool ellipsis, tree expr) { if (TYPE_QUALS (TREE_TYPE (b->decl)) != TYPE_UNQUALIFIED || C_DECL_REGISTER (b->decl)) - error ("% as only parameter may not be qualified"); + error_at (b->locus, "% as only parameter may not be qualified"); /* There cannot be an ellipsis. */ if (ellipsis) - error ("% must be the only parameter"); + error_at (b->locus, "% must be the only parameter"); arg_info->types = void_list_node; return arg_info; @@ -6946,13 +6946,14 @@ get_parm_info (bool ellipsis, tree expr) /* Check for forward decls that never got their actual decl. */ if (TREE_ASM_WRITTEN (decl)) - error ("parameter %q+D has just a forward declaration", decl); + error_at (b->locus, + "parameter %q+D has just a forward declaration", decl); /* Check for (..., void, ...) and issue an error. */ else if (VOID_TYPE_P (type) && !DECL_NAME (decl)) { if (!gave_void_only_once_err) { - error ("% must be the only parameter"); + error_at (b->locus, "% must be the only parameter"); gave_void_only_once_err = true; } } @@ -6991,13 +6992,13 @@ get_parm_info (bool ellipsis, tree expr) { if (b->id) /* The %s will be one of 'struct', 'union', or 'enum'. */ - warning_at (input_location, 0, + warning_at (b->locus, 0, "%<%s %E%> declared inside parameter list" " will not be visible outside of this definition or" " declaration", keyword, b->id); else /* The %s will be one of 'struct', 'union', or 'enum'. */ - warning_at (input_location, 0, + warning_at (b->locus, 0, "anonymous %s declared inside parameter list" " will not be visible outside of this definition or" " declaration", keyword); -- cgit v1.1