[Mesa-dev] [PATCH] i965/fs: Drop no-op shifts by 0.

Matt Turner mattst88 at gmail.com
Thu Oct 24 22:57:21 CEST 2013


On Thu, Oct 24, 2013 at 11:12 AM, Erik Faye-Lund <kusmabite at gmail.com> wrote:
> But wait a minute. x shifted by 0 is x, so the latter part looks
> correct. But the first conditional seems to assume that 0 sifted by x
> is x, but it's really 0, no? Shouldn't both cases return
> ir->operands[0]? What am I missing?

I think Eric and I should now be banned from reviewing each other's
patches to opt_algebraic. :)


More information about the mesa-dev mailing list