Message ID | 87612stg1u.fsf@kepler.schwinge.homeip.net |
---|---|
State | New |
Headers | show |
On 30/09/15 11:25, Thomas Schwinge wrote: > Don't we also want to commit the following change, which was part of your > trunk r227103 (883f001d2c3672e0674bec71f36a2052734a72cf) commit (and now > shows up as a delta between trunk and gomp-4_0-branch)? > > --- gcc/tree-cfg.c > +++ gcc/tree-cfg.c > @@ -6424,9 +6424,6 @@ replace_ssa_name (tree name, hash_map<tree, tree> *vars_map, > replace_by_duplicate_decl (&decl, vars_map, to_context); > new_name = make_ssa_name_fn (DECL_STRUCT_FUNCTION (to_context), > decl, SSA_NAME_DEF_STMT (name)); > - if (SSA_NAME_IS_DEFAULT_DEF (name)) > - set_ssa_default_def (DECL_STRUCT_FUNCTION (to_context), > - decl, new_name); > } > else > new_name = copy_ssa_name_fn (DECL_STRUCT_FUNCTION (to_context), Indeed, that bit is part of the patch "Don't create superfluous parm in expand_omp_taskreg", but was dropped in the merge (probably because it conflicted with the "Fix release_dangling_ssa_names" patch that I just reverted). So we need to apply it. Thanks, - Tom
--- gcc/tree-cfg.c +++ gcc/tree-cfg.c @@ -6424,9 +6424,6 @@ replace_ssa_name (tree name, hash_map<tree, tree> *vars_map, replace_by_duplicate_decl (&decl, vars_map, to_context); new_name = make_ssa_name_fn (DECL_STRUCT_FUNCTION (to_context), decl, SSA_NAME_DEF_STMT (name)); - if (SSA_NAME_IS_DEFAULT_DEF (name)) - set_ssa_default_def (DECL_STRUCT_FUNCTION (to_context), - decl, new_name); } else new_name = copy_ssa_name_fn (DECL_STRUCT_FUNCTION (to_context),