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