This is the mail archive of the gcc-patches@gcc.gnu.org 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,i386] fix PR 11001

On Fri, Oct 12, 2007 at 10:13:23AM -0700, Nathan Froyd wrote:
> On Fri, Oct 12, 2007 at 09:40:27AM -0700, H.J. Lu wrote:
> > On Fri, Oct 12, 2007 at 09:12:47AM -0700, Nathan Froyd wrote:
> > > +      /* Can't use this if the user has appropriated eax, ecx, or edi.  */
> > > +      if (global_regs[0] || global_regs[2] || global_regs[5])
> > > +        return false;
> > > +
> >
> > Can you use/add AX_REG, CX_REG, DI_REG, SI_REG instead using 0, 2,                                       
> > 4, 5?
> 
> Sure.  Patch below.  OK to commit if it passes bootstrap/testing?
> 
> -Nathan
> 
> 2007-10-12  Nathan Froyd  <froydnj@codesourcery.com>
> 
> 	* config/i386/i386.md (SI_REG, DI_REG): New constants.
> 	(strmov): Use defined constants.
> 	(cmpstrnsi): Likewise.
> 	* config/i386/i386.c (decide_alg): Use defined constants.
> 	(ix86_expand_strlen): Likewise.
> 

Did you use the old ChangeLog entries for checkin? I didn't
see SI_REG, DI_REG in ChangeLog.


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