[Mesa-dev] [PATCH] i965/fs: Remove try_replace_with_sel().

[Jason Ekstrand]

On Fri, Dec 5, 2014 at 10:24 AM, Matt Turner <mattst88 at gmail.com> wrote:

> On Fri, Dec 5, 2014 at 7:02 AM, Jason Ekstrand <jason at jlekstrand.net>
> wrote:
> > First off, how is this different from the sel peephole?
>
> This happens in the visitor during translation from GLSL IR to FS IR.
> It only looks for an exact sequence of IF/MOV/ELSE/MOV/ENDIF where the
> MOVs are to the same destination.
>
> The peephole happens in the optimization loop itself. The peephole
> looks for sequences of MOVs, rather than just a single one. The
> peephole also emits a MOV if the two sources you're selecting from are
> identical.
>

So why isn't the peephole picking up on what the other one is?  I'm
confused as to why we have any shaderdb change at all.  Any insight on
that?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20141205/2309d6cb/attachment.html>