| Index: gcc/gcc/tree-ssa-copyrename.c
|
| diff --git a/gcc/gcc/tree-ssa-copyrename.c b/gcc/gcc/tree-ssa-copyrename.c
|
| index d30e2372e7e5e5d92633d793c0ac0682808a267c..d62a09416d14da5a44f3393c1112baaa8e74ac72 100644
|
| --- a/gcc/gcc/tree-ssa-copyrename.c
|
| +++ b/gcc/gcc/tree-ssa-copyrename.c
|
| @@ -50,22 +50,22 @@ along with GCC; see the file COPYING3. If not see
|
| T.3_5 = <blah>
|
| a_1 = T.3_5
|
|
|
| - If this copy couldn't be copy propagated, it could possibly remain in the
|
| - program throughout the optimization phases. After SSA->normal, it would
|
| + If this copy couldn't be copy propagated, it could possibly remain in the
|
| + program throughout the optimization phases. After SSA->normal, it would
|
| become:
|
|
|
| T.3 = <blah>
|
| a = T.3
|
| -
|
| - Since T.3_5 is distinct from all other SSA versions of T.3, there is no
|
| - fundamental reason why the base variable needs to be T.3, subject to
|
| - certain restrictions. This optimization attempts to determine if we can
|
| +
|
| + Since T.3_5 is distinct from all other SSA versions of T.3, there is no
|
| + fundamental reason why the base variable needs to be T.3, subject to
|
| + certain restrictions. This optimization attempts to determine if we can
|
| change the base variable on copies like this, and result in code such as:
|
|
|
| a_5 = <blah>
|
| a_1 = a_5
|
|
|
| - This gives the SSA->normal pass a shot at coalescing a_1 and a_5. If it is
|
| + This gives the SSA->normal pass a shot at coalescing a_1 and a_5. If it is
|
| possible, the copy goes away completely. If it isn't possible, a new temp
|
| will be created for a_5, and you will end up with the exact same code:
|
|
|
| @@ -79,8 +79,8 @@ along with GCC; see the file COPYING3. If not see
|
| a_1 = <blah>
|
| <blah2> = a_1
|
|
|
| - get turned into
|
| -
|
| + get turned into
|
| +
|
| T.3_5 = <blah>
|
| a_1 = T.3_5
|
| <blah2> = a_1
|
| @@ -99,7 +99,7 @@ along with GCC; see the file COPYING3. If not see
|
| <blah2> = a_1
|
|
|
| which copy propagation would then turn into:
|
| -
|
| +
|
| a_5 = <blah>
|
| <blah2> = a_5
|
|
|
| @@ -115,7 +115,6 @@ copy_rename_partition_coalesce (var_map map, tree var1, tree var2, FILE *debug)
|
| int p1, p2, p3;
|
| tree root1, root2;
|
| tree rep1, rep2;
|
| - var_ann_t ann1, ann2, ann3;
|
| bool ign1, ign2, abnorm;
|
|
|
| gcc_assert (TREE_CODE (var1) == SSA_NAME);
|
| @@ -144,9 +143,6 @@ copy_rename_partition_coalesce (var_map map, tree var1, tree var2, FILE *debug)
|
| root1 = SSA_NAME_VAR (rep1);
|
| root2 = SSA_NAME_VAR (rep2);
|
|
|
| - ann1 = var_ann (root1);
|
| - ann2 = var_ann (root2);
|
| -
|
| if (p1 == p2)
|
| {
|
| if (debug)
|
| @@ -191,7 +187,7 @@ copy_rename_partition_coalesce (var_map map, tree var1, tree var2, FILE *debug)
|
| ign1 = TREE_CODE (root1) == VAR_DECL && DECL_IGNORED_P (root1);
|
| ign2 = TREE_CODE (root2) == VAR_DECL && DECL_IGNORED_P (root2);
|
|
|
| - /* Never attempt to coalesce 2 user variables unless one is an inline
|
| + /* Never attempt to coalesce 2 user variables unless one is an inline
|
| variable. */
|
| if (!ign1 && !ign2)
|
| {
|
| @@ -199,7 +195,7 @@ copy_rename_partition_coalesce (var_map map, tree var1, tree var2, FILE *debug)
|
| ign2 = true;
|
| else if (DECL_FROM_INLINE (root1))
|
| ign1 = true;
|
| - else
|
| + else
|
| {
|
| if (debug)
|
| fprintf (debug, " : 2 different USER vars. No coalesce.\n");
|
| @@ -207,17 +203,7 @@ copy_rename_partition_coalesce (var_map map, tree var1, tree var2, FILE *debug)
|
| }
|
| }
|
|
|
| - /* Don't coalesce if there are two different memory tags. */
|
| - if (ann1->symbol_mem_tag
|
| - && ann2->symbol_mem_tag
|
| - && ann1->symbol_mem_tag != ann2->symbol_mem_tag)
|
| - {
|
| - if (debug)
|
| - fprintf (debug, " : 2 memory tags. No coalesce.\n");
|
| - return false;
|
| - }
|
| -
|
| - /* If both values have default defs, we can't coalesce. If only one has a
|
| + /* If both values have default defs, we can't coalesce. If only one has a
|
| tag, make sure that variable is the new root partition. */
|
| if (gimple_default_def (cfun, root1))
|
| {
|
| @@ -247,42 +233,20 @@ copy_rename_partition_coalesce (var_map map, tree var1, tree var2, FILE *debug)
|
| return false;
|
| }
|
|
|
| - /* Don't coalesce if the aliasing sets of the types are different. */
|
| - if (POINTER_TYPE_P (TREE_TYPE (root1))
|
| - && POINTER_TYPE_P (TREE_TYPE (root2))
|
| - && ((get_alias_set (TREE_TYPE (TREE_TYPE (root1)))
|
| - != get_alias_set (TREE_TYPE (TREE_TYPE (root2))))
|
| - || ((DECL_P (root1) && !MTAG_P (root1))
|
| - && (DECL_P (root2) && !MTAG_P (root2))
|
| - && DECL_NO_TBAA_P (root1) != DECL_NO_TBAA_P (root2))))
|
| - {
|
| - if (debug)
|
| - fprintf (debug, " : 2 different aliasing sets. No coalesce.\n");
|
| - return false;
|
| - }
|
| -
|
| -
|
| /* Merge the two partitions. */
|
| p3 = partition_union (map->var_partition, p1, p2);
|
|
|
| - /* Set the root variable of the partition to the better choice, if there is
|
| + /* Set the root variable of the partition to the better choice, if there is
|
| one. */
|
| if (!ign2)
|
| replace_ssa_name_symbol (partition_to_var (map, p3), root2);
|
| else if (!ign1)
|
| replace_ssa_name_symbol (partition_to_var (map, p3), root1);
|
|
|
| - /* Update the various flag widgitry of the current base representative. */
|
| - ann3 = var_ann (SSA_NAME_VAR (partition_to_var (map, p3)));
|
| - if (ann1->symbol_mem_tag)
|
| - ann3->symbol_mem_tag = ann1->symbol_mem_tag;
|
| - else
|
| - ann3->symbol_mem_tag = ann2->symbol_mem_tag;
|
| -
|
| if (debug)
|
| {
|
| fprintf (debug, " --> P%d ", p3);
|
| - print_generic_expr (debug, SSA_NAME_VAR (partition_to_var (map, p3)),
|
| + print_generic_expr (debug, SSA_NAME_VAR (partition_to_var (map, p3)),
|
| TDF_SLIM);
|
| fprintf (debug, "\n");
|
| }
|
| @@ -292,8 +256,8 @@ copy_rename_partition_coalesce (var_map map, tree var1, tree var2, FILE *debug)
|
|
|
| /* This function will make a pass through the IL, and attempt to coalesce any
|
| SSA versions which occur in PHI's or copies. Coalescing is accomplished by
|
| - changing the underlying root variable of all coalesced version. This will
|
| - then cause the SSA->normal pass to attempt to coalesce them all to the same
|
| + changing the underlying root variable of all coalesced version. This will
|
| + then cause the SSA->normal pass to attempt to coalesce them all to the same
|
| variable. */
|
|
|
| static unsigned int
|
| @@ -313,7 +277,7 @@ rename_ssa_copies (void)
|
| else
|
| debug = NULL;
|
|
|
| - map = init_var_map (num_ssa_names + 1);
|
| + map = init_var_map (num_ssa_names);
|
|
|
| FOR_EACH_BB (bb)
|
| {
|
| @@ -360,13 +324,13 @@ rename_ssa_copies (void)
|
|
|
| /* Now one more pass to make all elements of a partition share the same
|
| root variable. */
|
| -
|
| - for (x = 1; x <= num_ssa_names; x++)
|
| +
|
| + for (x = 1; x < num_ssa_names; x++)
|
| {
|
| part_var = partition_to_var (map, x);
|
| if (!part_var)
|
| continue;
|
| - var = map->partition_to_var[x];
|
| + var = ssa_name (x);
|
| if (debug)
|
| {
|
| if (SSA_NAME_VAR (var) != SSA_NAME_VAR (part_var))
|
| @@ -393,7 +357,7 @@ gate_copyrename (void)
|
| return flag_tree_copyrename != 0;
|
| }
|
|
|
| -struct gimple_opt_pass pass_rename_ssa_copies =
|
| +struct gimple_opt_pass pass_rename_ssa_copies =
|
| {
|
| {
|
| GIMPLE_PASS,
|
| @@ -407,7 +371,7 @@ struct gimple_opt_pass pass_rename_ssa_copies =
|
| PROP_cfg | PROP_ssa, /* properties_required */
|
| 0, /* properties_provided */
|
| 0, /* properties_destroyed */
|
| - 0, /* todo_flags_start */
|
| + 0, /* todo_flags_start */
|
| TODO_dump_func | TODO_verify_ssa /* todo_flags_finish */
|
| }
|
| -};
|
| +};
|
|
|