This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: remove C_EXPR_APPEND macro
- From: Prathamesh Kulkarni <bilbotheelffriend at gmail dot com>
- To: "Joseph S. Myers" <joseph at codesourcery dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Sat, 8 Feb 2014 02:51:03 +0530
- Subject: Re: remove C_EXPR_APPEND macro
- Authentication-results: sourceware.org; auth=none
- References: <CAJXstsBgaw5gz3UYfO-UbQ=61Xvw5jvi8ngkkqAQxPtFFbwhVA at mail dot gmail dot com> <Pine dot LNX dot 4 dot 64 dot 1402072049340 dot 12232 at digraph dot polyomino dot org dot uk>
On Sat, Feb 8, 2014 at 2:22 AM, Joseph S. Myers <joseph@codesourcery.com> wrote:
> On Sat, 8 Feb 2014, Prathamesh Kulkarni wrote:
>
>> This patch removes C_EXPR_APPEND macro in c-tree.h
>> OK for trunk ?
>
> Thanks, this is OK with the orphan comment "A varray of c_expr_t." also
> removed (please send the revised patch if you'd like someone to commit it
> for you).
Ah, I missed that, sorry.
Here's the revised patch:
* c-parser.c (c_parser_get_builtin_args): replace calls to
C_EXPR_APPEND (cexpr_list, expr) by vec_safe_push (cexpr_list, expr)
* c-tree.h (C_EXPR_APPEND): removed
Index: gcc/c/c-parser.c
===================================================================
--- gcc/c/c-parser.c (revision 207610)
+++ gcc/c/c-parser.c (working copy)
@@ -6659,12 +6659,12 @@ c_parser_get_builtin_args (c_parser *par
force_folding_builtin_constant_p
= saved_force_folding_builtin_constant_p;
vec_alloc (cexpr_list, 1);
- C_EXPR_APPEND (cexpr_list, expr);
+ vec_safe_push (cexpr_list, expr);
while (c_parser_next_token_is (parser, CPP_COMMA))
{
c_parser_consume_token (parser);
expr = c_parser_expr_no_commas (parser, NULL);
- C_EXPR_APPEND (cexpr_list, expr);
+ vec_safe_push (cexpr_list, expr);
}
if (!c_parser_require (parser, CPP_CLOSE_PAREN, "expected %<)%>"))
Index: gcc/c/c-tree.h
===================================================================
--- gcc/c/c-tree.h (revision 207610)
+++ gcc/c/c-tree.h (working copy)
@@ -132,15 +132,6 @@ struct c_expr
inside the VEC types. */
typedef struct c_expr c_expr_t;
-/* A varray of c_expr_t. */
-
-/* Append a new c_expr_t element to V. */
-#define C_EXPR_APPEND(V, ELEM) \
- do { \
- c_expr_t __elem = (ELEM); \
- vec_safe_push (V, __elem); \
- } while (0)
-
/* A kind of type specifier. Note that this information is currently
only used to distinguish tag definitions, tag references and typeof
uses. */
>
> Although this is small enough to go in without a copyright assignment,
> it's a good idea to start on the paperwork if you might be making larger
> changes in future.
>
> http://git.savannah.gnu.org/cgit/gnulib.git/plain/doc/Copyright/request-assign.future
Thanks
>
> --
> Joseph S. Myers
> joseph@codesourcery.com