This is the mail archive of the binutils@sources.redhat.com mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Add .set arch=FOO support to MIPS gas.


cgd@broadcom.com wrote:
[snip]
> > But they are semantically different. arch= changes the ISA, but
> > mipsX does not change the architecture.
> 
> right, what i meant was something like:
> 
>         if (mipsN or arch=) {
>           if (mips0)
>                 set ISA
>           else if (mips1)
>                 ...
>           else if (arch=)
>                 handle mipsN, i.e. set arch and ISA
>           else
>                 bugs mipsN values.
>           switch on ISA to set gp32/gp64 as is there now.
>         }
> 
> hmm, here's a question (meant to ask this before)...
> 
> should ".set mips0" reset the *arch* to the default, too?
> 
> (i don't see any *harm* in it, and then both .set mips0 and .set
> arch=default would restore arch/isa sanity?)

It would disable CPU special opcodes then, which is a behaviour change
and probably confusing. So it shouldn't change IMHO.


Thiemo


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]