Patchwork Remove call_expr_arg and call_expr_argp

login
register
mail settings
Submitter Romain Geissler
Date July 8, 2011, 8:44 a.m.
Message ID <CAF+LTecXhsAhsxEB=a5Y2t6qV0S6nww3m4pJ9SPqare5bbDirg@mail.gmail.com>
Download mbox | patch
Permalink /patch/103783/
State New
Headers show

Comments

Romain Geissler - July 8, 2011, 8:44 a.m.
Hello,

This patch simply removes legacy call_expr_arg and call_expr_argp
declarations since these functions no longer have a defined body. Use
CALL_EXPR_ARG and CALL_EXPR_ARGP macros instead.

Build fine and no regression on x86_64.

2011-07-08  Romain Geissler  <romain.geissler@gmail.com>

       * tree.h: Remove call_expr_arg and call_expr_argp
Richard Guenther - July 8, 2011, 8:45 a.m.
On Fri, Jul 8, 2011 at 10:44 AM, Romain Geissler
<romain.geissler@gmail.com> wrote:
> Hello,
>
> This patch simply removes legacy call_expr_arg and call_expr_argp
> declarations since these functions no longer have a defined body. Use
> CALL_EXPR_ARG and CALL_EXPR_ARGP macros instead.
>
> Build fine and no regression on x86_64.

Ok.

Thanks,
Richard.

> 2011-07-08  Romain Geissler  <romain.geissler@gmail.com>
>
>       * tree.h: Remove call_expr_arg and call_expr_argp
>
>
>
> Index: gcc/tree.h
> ===================================================================
> --- gcc/tree.h  (revision 175907)
> +++ gcc/tree.h  (working copy)
> @@ -5130,8 +5130,6 @@ extern bool commutative_ternary_tree_cod
>  extern tree upper_bound_in_type (tree, tree);
>  extern tree lower_bound_in_type (tree, tree);
>  extern int operand_equal_for_phi_arg_p (const_tree, const_tree);
> -extern tree call_expr_arg (tree, int);
> -extern tree *call_expr_argp (tree, int);
>  extern tree create_artificial_label (location_t);
>  extern const char *get_name (tree);
>  extern bool stdarg_p (const_tree);
>
Romain Geissler - July 11, 2011, 7:53 a.m.
2011/7/8 Richard Guenther <richard.guenther@gmail.com>:
> Ok.
>
> Thanks,
> Richard.
>

Hi,

I don't have write access, can you please add the patch to the trunk ?

Romain Geissler
Richard Guenther - July 11, 2011, 10:01 a.m.
On Mon, Jul 11, 2011 at 9:53 AM, Romain Geissler
<romain.geissler@gmail.com> wrote:
> 2011/7/8 Richard Guenther <richard.guenther@gmail.com>:
>> Ok.
>>
>> Thanks,
>> Richard.
>>
>
> Hi,
>
> I don't have write access, can you please add the patch to the trunk ?

Done.  Btw, a proper changelog would have been

2011-07-11  Romain Geissler  <romain.geissler@gmail.com>

       * tree.h (call_expr_arg): Remove.
       (call_expr_argp): Likewise.


> Romain Geissler
>

Patch

Index: gcc/tree.h
===================================================================
--- gcc/tree.h  (revision 175907)
+++ gcc/tree.h  (working copy)
@@ -5130,8 +5130,6 @@  extern bool commutative_ternary_tree_cod
 extern tree upper_bound_in_type (tree, tree);
 extern tree lower_bound_in_type (tree, tree);
 extern int operand_equal_for_phi_arg_p (const_tree, const_tree);
-extern tree call_expr_arg (tree, int);
-extern tree *call_expr_argp (tree, int);
 extern tree create_artificial_label (location_t);
 extern const char *get_name (tree);
 extern bool stdarg_p (const_tree);