Re: [Mesa-dev] [PATCH v2 6/6] nv50/ir: teach insnCanLoad() about SHLADD

2016-09-26 Thread Ilia Mirkin
Reviewed-by: Ilia Mirkin On Mon, Sep 26, 2016 at 5:02 PM, Samuel Pitoiset wrote: > Commutativity is not allowed with SHLADD, but src2 can accept > loads. To allow the load propagation pass to do its job, add a > special case like for SUCLAMP because src1 is always an immediate. > > This IMAD to

[Mesa-dev] [PATCH v2 6/6] nv50/ir: teach insnCanLoad() about SHLADD

2016-09-26 Thread Samuel Pitoiset
Commutativity is not allowed with SHLADD, but src2 can accept loads. To allow the load propagation pass to do its job, add a special case like for SUCLAMP because src1 is always an immediate. This IMAD to SHLADD optimization helps a bunch of shaders from Tomb Raider, Victor Vran, UE4 demos (+15% p