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

Reply via email to