mips64-linux-gnuabi64 testsuite breakage
Maciej W. Rozycki
macro@orcam.me.uk
Tue May 2 19:33:00 GMT 2023
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), but it hasn't
been properly verified either, as a change is supposed not to cause
regressions in the testsuite.
Please revert the part quoted and fix the regressions. Thank you.
Maciej
More information about the Binutils
mailing list