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