From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by sourceware.org (Postfix) with ESMTP id A8D2C3851A9A for ; Wed, 31 Aug 2022 08:20:06 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org A8D2C3851A9A Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=arm.com Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 2F5F01042; Wed, 31 Aug 2022 01:20:12 -0700 (PDT) Received: from localhost (e121540-lin.manchester.arm.com [10.32.98.62]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 886C23F766; Wed, 31 Aug 2022 01:20:05 -0700 (PDT) From: Richard Sandiford To: gcc-patches@gcc.gnu.org Mail-Followup-To: gcc-patches@gcc.gnu.org,jakub@redhat.com, richard.sandiford@arm.com Cc: jakub@redhat.com Subject: [pushed] vect: Fix stray argument in call to dump_printf_loc Date: Wed, 31 Aug 2022 09:20:04 +0100 Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Spam-Status: No, score=-49.8 required=5.0 tests=BAYES_00,GIT_PATCH_0,KAM_DMARC_NONE,KAM_DMARC_STATUS,KAM_LAZY_DOMAIN_SECURITY,SPF_HELO_NONE,SPF_NONE,TXREP,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: 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