Hi all,

just a small patch removing a piece of unreachable code in
'vect_estimate_min_profitable_iters' given the condition
(LOOP_VINFO_USING_PARTIAL_VECTORS_P (loop_vinfo)) is always true as
checked just above.
 
Bootstrapped on aarch64-unknown-linux-gnu.

Okay for trunk?

  Andrea

>From 6bd96581410d9847ab78e6761178d4efbc9a5af2 Mon Sep 17 00:00:00 2001
From: Andrea Corallo <andrea.cora...@arm.com>
Date: Fri, 4 Sep 2020 09:56:59 +0100
Subject: [PATCH] vec: dead code removal in tree-vect-loop.c

gcc/ChangeLog

2020-09-04  Andrea Corallo  <andrea.cora...@arm.com>

        * tree-vect-loop.c (vect_estimate_min_profitable_iters): Remove
        dead code as LOOP_VINFO_USING_PARTIAL_VECTORS_P (loop_vinfo) is
        always verified.
---
 gcc/tree-vect-loop.c | 15 ++++-----------
 1 file changed, 4 insertions(+), 11 deletions(-)

diff --git a/gcc/tree-vect-loop.c b/gcc/tree-vect-loop.c
index 362cdc4f1cb..38576382fe7 100644
--- a/gcc/tree-vect-loop.c
+++ b/gcc/tree-vect-loop.c
@@ -4101,17 +4101,10 @@ vect_estimate_min_profitable_iters (loop_vec_info 
loop_vinfo,
       if (outside_overhead > 0)
        min_vec_niters = outside_overhead / saving_per_viter + 1;
 
-      if (LOOP_VINFO_USING_PARTIAL_VECTORS_P (loop_vinfo))
-       {
-         int threshold = (vec_inside_cost * min_vec_niters
-                          + vec_outside_cost
-                          + scalar_outside_cost);
-         min_profitable_estimate = threshold / scalar_single_iter_cost + 1;
-       }
-      else
-       min_profitable_estimate = (min_vec_niters * assumed_vf
-                                  + peel_iters_prologue
-                                  + peel_iters_epilogue);
+      int threshold = (vec_inside_cost * min_vec_niters
+                      + vec_outside_cost
+                      + scalar_outside_cost);
+      min_profitable_estimate = threshold / scalar_single_iter_cost + 1;
     }
   else
     {
-- 
2.17.1

Reply via email to