Remove unnecessary temporary in tree-if-conv.c

The call to ifc_temp_var in predicate_mem_writes become redundant
in r230099.  Before that point the mask was calculated using
fold_build_*s, but now it's calculated by gimple_build and so
is already a valid gimple value.

As it stands, the call forces an SSA_NAME-to-SSA_NAME copy
to be created, whereas SLP expects that such redundant copies
have already been eliminated.

2018-01-13  Richard Sandiford  <richard.sandiford@linaro.org>

gcc/
	* tree-if-conv.c (predicate_mem_writes): Remove redundant
	call to ifc_temp_var.

From-SVN: r256638

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index d855951151a5d7082306a47ece8d21ba6c15b6b5..2edd769c58827cdaf3fdbc323209864989f49d89 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2018-01-13  Richard Sandiford  <richard.sandiford@linaro.org>
+
+       * tree-if-conv.c (predicate_mem_writes): Remove redundant
+       call to ifc_temp_var.
+
 2018-01-13  Richard Sandiford  <richard.sandiford@linaro.org>
            Alan Hayward  <alan.hayward@arm.com>
            David Sherwood  <david.sherwood@arm.com>

diff --git a/gcc/tree-if-conv.c b/gcc/tree-if-conv.c
index 4f849537beef245a204336072076a12a52bacac9..cac3fd75d548d499fd4bd8645e84a9d795be2846 100644 (file)
--- a/gcc/tree-if-conv.c
+++ b/gcc/tree-if-conv.c
@@ -2261,7 +2261,6 @@ predicate_mem_writes (loop_p loop)
                    }
                  gsi_insert_seq_before (&gsi, stmts, GSI_SAME_STMT);
 
-                 mask = ifc_temp_var (TREE_TYPE (mask), mask, &gsi);
                  /* Save mask and its size for further use.  */
                  vect_sizes.safe_push (bitsize);
                  vect_masks.safe_push (mask);