[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