From b89fa419ca39b13b5ed0f7a23722b394b3af399e Mon Sep 17 00:00:00 2001 From: Richard Sandiford Date: Sat, 13 Jan 2018 18:01:14 +0000 Subject: [PATCH] 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 gcc/ * tree-if-conv.c (predicate_mem_writes): Remove redundant call to ifc_temp_var. From-SVN: r256638 --- gcc/ChangeLog | 5 +++++ gcc/tree-if-conv.c | 1 - 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index d855951151a..2edd769c588 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2018-01-13 Richard Sandiford + + * tree-if-conv.c (predicate_mem_writes): Remove redundant + call to ifc_temp_var. + 2018-01-13 Richard Sandiford Alan Hayward David Sherwood diff --git a/gcc/tree-if-conv.c b/gcc/tree-if-conv.c index 4f849537bee..cac3fd75d54 100644 --- 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); -- 2.30.2