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

Matt Turner mattst88 at gmail.com
Tue Jan 13 13:08:37 PST 2015


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.

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.

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>


More information about the mesa-dev mailing list