This avoids dumping 'vectorization is not profitable' one more time
if none of the opportunities in a BB is profitable.

tested on x86_64-unknown-linux-gnu, pushed.

2020-09-11  Richard Biener  <rguent...@suse.de>

        PR tree-optimization/97013
        * tree-vect-slp.c (vect_slp_analyze_bb_1): Remove duplicate dumping.
---
 gcc/tree-vect-slp.c | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/gcc/tree-vect-slp.c b/gcc/tree-vect-slp.c
index aa6aa3328e8..35bde9bcb9d 100644
--- a/gcc/tree-vect-slp.c
+++ b/gcc/tree-vect-slp.c
@@ -3626,16 +3626,10 @@ vect_slp_analyze_bb_1 (bb_vec_info bb_vinfo, int 
n_stmts, bool &fatal,
 
   vect_bb_partition_graph (bb_vinfo);
 
-  /* Cost model: check if the vectorization is worthwhile.  */
+  /* Cost model: check if the vectorization opportunities are worthwhile.  */
   if (!unlimited_cost_model (NULL)
       && !vect_bb_vectorization_profitable_p (bb_vinfo))
-    {
-      if (dump_enabled_p ())
-        dump_printf_loc (MSG_MISSED_OPTIMIZATION, vect_location,
-                        "not vectorized: vectorization is not "
-                        "profitable.\n");
-      return false;
-    }
+    return false;
 
   if (dump_enabled_p ())
     dump_printf_loc (MSG_NOTE, vect_location,
-- 
2.26.2

Reply via email to