[PATCH] MIPS gas: Remove superfluous CPU_* checks
Thiemo Seufer
ica2_ts@csv.ica.uni-stuttgart.de
Fri Apr 23 04:20:00 GMT 2004
cgd@broadcom.com wrote:
> At Fri, 23 Apr 2004 01:18:48 +0000 (UTC), "Thiemo Seufer" wrote:
> > I applied the appended patch. Those CPUs are already covered
> > by checking their ISA.
>
> given the behaviour of the '.set' commands, is that necessarily true?
You are right, it's not true. But for .set arch=FOO it should be ok, as
it sets arch and isa.
For .set mipsN we lose the hilo_interlocks optimization for SB1 if a
MIPS IV or lower ISA is chosen.
We lose gpr_interlocks optimizations for VR5400, VR5500 with .set mips1.
We lose cop_interlocks optimizations for VR5400, VR5500, SB1 with
.set mips[1-3]
I think dropping the optimization when the ISA level was explicitly
lowered is an improvement. We should probably generally set the arch
to the default value of the .set ISA.
Thiemo
More information about the Binutils
mailing list