One call to dump_printf_loc had a stray left-over argument
from an earlier version of the patch.  This went unnoticed
on aarch64-linux-gnu and x86_64-linux-gnu since the parameters
that actually mattered were passed in FPRs rather than GPRs,
but I assume this is the reason for the i686-linux-gnu failures
that Jakub hit.

Tested on x86_64-linux-gnu & pushed as obvious.  Sorry for the breakage.

Richard


gcc/
        * tree-vect-slp.cc (vect_optimize_slp_pass::dump): Remove bogus
        argument.
---
 gcc/tree-vect-slp.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gcc/tree-vect-slp.cc b/gcc/tree-vect-slp.cc
index ab4c6fa6776..226550635cc 100644
--- a/gcc/tree-vect-slp.cc
+++ b/gcc/tree-vect-slp.cc
@@ -5461,7 +5461,7 @@ vect_optimize_slp_pass::dump ()
              combined_cost.add_serial_cost (layout_costs.out_cost);
 #define TEMPLATE "{depth: %f, total: %f}"
              dump_printf_loc (MSG_NOTE, vect_location,
-                              "        " TEMPLATE "\n", layout_i,
+                              "        " TEMPLATE "\n",
                               layout_costs.in_cost.depth.to_double (),
                               layout_costs.in_cost.total.to_double ());
              dump_printf_loc (MSG_NOTE, vect_location,
-- 
2.25.1

Reply via email to