On Tue, Mar 29, 2016 at 07:21:29PM +0200, Jakub Jelinek wrote:
> This is a case similar to the LSHIFTRT I've fixed recently.
> But, unlike LSHIFTRT, which can be handled by masking at the outer level,
> ASHIFTRT would need outer sign extension, so most of the time 2 outer
> operations in addition to the kept two inner shifts, which is IMHO very
> unlikely to ever be successfully combined on any target nor actually
> beneficial.  So this patch just avoids that optimization for ASHIFTRT
> if there are different modes.
> 
> Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?

Okay with spello fixed ("queuing").  Thanks,


Segher

Reply via email to