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 5D1D43858C53 for ; Tue, 9 May 2023 08:13:10 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 5D1D43858C53 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 9424B1063; Tue, 9 May 2023 01:13:54 -0700 (PDT) Received: from localhost (e121540-lin.manchester.arm.com [10.32.110.72]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 390053F67D; Tue, 9 May 2023 01:13:09 -0700 (PDT) From: Richard Sandiford To: "Maciej W. Rozycki" Mail-Followup-To: "Maciej W. Rozycki" ,YunQiang Su , Alan Modra , binutils@sourceware.org, YunQiang Su , Chenghua Xu , richard.sandiford@arm.com Cc: YunQiang Su , Alan Modra , binutils@sourceware.org, YunQiang Su , Chenghua Xu Subject: Re: mips64-linux-gnuabi64 testsuite breakage References: Date: Tue, 09 May 2023 09:13:07 +0100 In-Reply-To: (Maciej W. Rozycki's message of "Tue, 2 May 2023 20:33:00 +0100 (BST)") 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=-24.0 required=5.0 tests=BAYES_00,KAM_DMARC_NONE,KAM_DMARC_STATUS,KAM_LAZY_DOMAIN_SECURITY,SPF_HELO_NONE,SPF_NONE,TXREP,T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: "Maciej W. Rozycki" writes: > On Wed, 26 Apr 2023, YunQiang Su wrote: > >> > Since 32f1c80375e "MIPS: support mips*64 as CPU and gnuabi64 as ABI", >> >> I will fix the test suite. >> The reason is that: the previous test suite assumes the default ABI is >> n32 even the triple >> with -gnuabi64. > > I can see extra 3346 regressions across various MIPS targets compared to > a checkout from Jan. Including ones like (parts of log trimmed for > clarity): > > ../as-new --defsym count=960 /scratch/vol1/binutils/binutils-mips-test/binutils-src/gas/testsuite/gas/mips/branch-swap-2.s > gas/testsuite/gas/mips/branch-swap-2.s:1: Fatal error: `micromips' cannot be used with `mips32r6' > FAIL: MIPS branch swapping (960) > > for `mips-img-elf' which indicates that the default architecture has > changed for GAS for this target. I think this is due to your commit > 9171de358f23 ("MIPS: default output r6 obj if the triple is r6"), which > has this part among others: > > + # If Vendor is IMG, then MIPSr6 is used > + case ${target} in > + mips*64*-img-*) > + mips_cpu=mips64r6 > + ;; > + mips*-img-*) > + mips_cpu=mips32r6 > + ;; > + esac > > Not only this change is wrong (you can't just arbitrarily change a > configuration that has been in the wild for ~8.5 years as it'll break > things for people who rely on the established semantics), It wasn't supposed to be an arbitrary change, but instead was supposed to sync GAS's default to GCC's. The GCC img toolchain has been an r6 toolchain since it was added in 2014. The fact that GAS instead defaults to mips1/mips3 seems like a bug, since mips1 isn't link-compatible with mips32r6 and mips3 isn't link-compatible with mips64r6. > but it hasn't been properly verified either, as a change is supposed > not to cause regressions in the testsuite. Can't argue with that part. But... > Please revert the part quoted and fix the regressions. Thank you. ...I think it does make sense to resurrect the patch in a testsuite-friendly form. Thanks, Richard