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] combine fix and allow rs6000 insvsi_internal4


On Thu, 2003-03-13 at 19:46, Richard Henderson wrote:
> On Thu, Mar 13, 2003 at 05:18:55PM -0800, Eric Christopher wrote:
> >    /* Don't eliminate a store in the stack pointer.  */
> >    if (dest == stack_pointer_rtx
> > -      /* If we couldn't eliminate a field assignment, we can't combine.  */
> > -      || GET_CODE (dest) == ZERO_EXTRACT || GET_CODE (dest) == STRICT_LOW_PART
> > +      || GET_CODE (dest) == STRICT_LOW_PART
> 
> STRICT_LOW_PART is a special case of ZERO_EXTRACT.  If the
> later works, the former should as well.
> 

OK. That was actually in my testing, and original patch. I'll go ahead
and remake the change.

Can anyone think of a reason why this shouldn't go in?

-eric

-- 
o/~ got caught stealing fire from the sky o/~


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