[Mesa-dev] [PATCH 036/133] i965/fs: Allow reinterpretation in constant propagation

[Jason Ekstrand]

On Tue, Jan 13, 2015 at 1:08 PM, Matt Turner <mattst88 at gmail.com> wrote:

> I would have no clue what this patch did if I just read the subject. How
> about:
>
> > i965/fs: Allow constant propagation between different types.
>

Sure.


>
> Also, this should go in separately from NIR, since it affects things
> (in Synmark) now:
>
> instructions in affected programs:     2288 -> 2144 (-6.29%)
>

> It allows us to copy propagate a 0x00000000UD into a 0.0F in a
> load_payload, which then lets us combine 4 load_payloads.
>

I'll add that and get it pushed.  Thanks for reviewing.


>
> On Mon, Dec 15, 2014 at 10:04 PM, Jason Ekstrand <jason at jlekstrand.net>
> wrote:
> > Because NIR is typeless, it treats all constants as uint32 values and
> > reinterprets them when they are used later.  This commit allows those
> > values to be properly propagated.
> > ---
>
> Reviewed-by: Matt Turner <mattst88 at gmail.com>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20150113/248d947d/attachment.html>