Re: PATCH: Add access restriction check for application registers (was Re: Is this a gas bug?)

["H. J. Lu" <hjl at lucon dot org>]

On Mon, May 24, 2004 at 11:15:59PM +0800, Jie Zhang wrote:
> On Mon, 2004-05-24 at 02:08, Andreas Schwab wrote:
> > Jie Zhang <jiez@citiz.net> writes:
> > 
> > > Gas puts these two alloc instruction into M-unit type slot without
> > > noting that ar.pfs can only be accessed by I-unit. I have no time to fix
> > > this now. I will try to fix it if nobody submit patch for it in about
> > > two weeks.
> > 
> > alloc surely is an M-unit insn.  The access restrictions about application
> > registers apply only to the mov insn.
> > 
> > Andreas.
> 
> Oh, my mistake! Here is the modified patch for this issue. Since the
> access restriction about application registers apply only to the mov
> insn, I move the check from emit_one_bundle () to md_assemble () and
> make it only check the first two operands of mov.i and mov.m. Now make
> check is OK. No need for another patch now:-)
> 

The patch is not corrrect. I am testing this patch now.


H.J.

Attachment: gas-ia64-ar-1.patch
Description: Text document