This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Make get_or_create_ssa_default_def consistently use the fn parameter
- From: Jakub Jelinek <jakub at redhat dot com>
- To: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 4 Mar 2013 18:17:20 +0100
- Subject: Re: [PATCH] Make get_or_create_ssa_default_def consistently use the fn parameter
- References: <20130304171123.GA4393@virgil.suse>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Mon, Mar 04, 2013 at 06:11:24PM +0100, Martin Jambor wrote:
> today I have noticed that get_or_create_ssa_default_def passes its
> struct function parameter to ssa_default_def but then happily uses a
> cfun-only make_ssa_name and even explicitely passes cfun to
> set_ssa_default_def. Fixed with the patch below.
>
> Bootstrapped and tested on x86_64-linux. Given that all current
> callers pass cfun as the first parameter I believe this is so low risk
> that I'd like to commit it to trunk even though it does not really
> cause any regression. However, I'm also fine with waiting for stage1.
> What do you think?
I'd prefer to see it in 4.8, please check it in. Thanks.
> 2013-03-04 Martin Jambor <mjambor@suse.cz>
>
> * tree-dfa.c (get_or_create_ssa_default_def): Use parameter fn in
> all appropriate places.
>
> Index: src/gcc/tree-dfa.c
> ===================================================================
> --- src.orig/gcc/tree-dfa.c
> +++ src/gcc/tree-dfa.c
> @@ -362,8 +362,8 @@ get_or_create_ssa_default_def (struct fu
> tree ddef = ssa_default_def (fn, var);
> if (ddef == NULL_TREE)
> {
> - ddef = make_ssa_name (var, gimple_build_nop ());
> - set_ssa_default_def (cfun, var, ddef);
> + ddef = make_ssa_name_fn (fn, var, gimple_build_nop ());
> + set_ssa_default_def (fn, var, ddef);
> }
> return ddef;
> }
Jakub