This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Remove unused nonlocal_value member of ipa_parm_adjustment
- From: Richard Biener <rguenther at suse dot de>
- To: Jakub Jelinek <jakub at redhat dot com>
- Cc: Martin Jambor <mjambor at suse dot cz>, gcc-patches at gcc dot gnu dot org
- Date: Wed, 13 Mar 2019 09:43:36 +0100 (CET)
- Subject: Re: [PATCH] Remove unused nonlocal_value member of ipa_parm_adjustment
- References: <20190313082032.GX7611@tucnak>
On Wed, 13 Mar 2019, Jakub Jelinek wrote:
> Hi!
>
> This field seems to have been unused since it has been added 10 years ago,
> the patch had just some comments about it in the code, but even those are
> long gone now. https://gcc.gnu.org/ml/gcc-patches/2009-07/msg00514.html
>
> Can we remove it now and only add if we ever need it for something?
> Bootstrapped/regtested on x86_64-linux and i686-linux.
You can remove it now.
Richard.
> 2019-03-13 Jakub Jelinek <jakub@redhat.com>
>
> * ipa-param-manipulation.h (struct ipa_parm_adjustment): Remove
> nonlocal_value member.
>
> --- gcc/ipa-param-manipulation.h.jj 2019-01-01 12:37:17.909962612 +0100
> +++ gcc/ipa-param-manipulation.h 2019-03-12 19:06:14.949156719 +0100
> @@ -75,10 +75,6 @@ struct ipa_parm_adjustment
> non-default-def ssa names when a parm decl is going away. */
> tree new_ssa_base;
>
> - /* If non-NULL and the original parameter is to be removed (copy_param below
> - is NULL), this is going to be its nonlocalized vars value. */
> - tree nonlocal_value;
> -
> /* This holds the prefix to be used for the new DECL_NAME. */
> const char *arg_prefix;
>
>
> Jakub
>
--
Richard Biener <rguenther@suse.de>
SUSE LINUX GmbH, GF: Felix Imendoerffer, Jane Smithard, Graham Norton, HRB 21284 (AG Nuernberg)