This is the mail archive of the mailing list for the GCC 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] MIPS: mips16e machine patterns - zeb/zeh seb/seh

On Fri, 2005-06-10 at 14:09 +0100, Richard Sandiford wrote:
> David Ung <> writes:
> > Looks like we've agreed to leave in the -mips16e option then.
> > here is a revised patch.  I've added a few lines in the manual and now
> > use GENERATE_MIPS16E instead, since TARGET_MIPS16E is already generated
> > by mips.opt.
> I'm a bit confused here.  I thought the two options under discussion were:
>   (a) -mips16e as a separate flag.  You get MIPS16e instructions
>       if and only if you use -mips16e.
>   (b) -mips16 is always used to enable MIPS16(e) instructions.
>       The -march setting decides whether MIPS16e is available.
> (a) was what your first patch did and (b) was what I mentioned in
> the review (purely for discussion purposes ;).
> Like you say, the balance of opinion was that having a separate
> flag was better than (b), but you now seem to have gone for a
> hybrid in which -mips16 sometimes selects MIPS16 and sometimes
> selects MIPS16e.  Why did you do that rather than stick with
> your original?

That's because to be MIPS32/32R2/64/64R2 ASE compliant, the full MIPS16e
set is required, not a subset (MIPS16). I've simply made that if the
user gave -mips16 option for mips_isa>=32, they really mean -mips16e.
As in Ian's previous reply, -march=mips32 -mips16 will imply -mips16e.


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