On Wednesday, August 29, 2018 10:11:48 AM PDT Jason Ekstrand wrote:
> This commit moves our storage image format conversion codegen into NIR
> instead of doing it in the back-end.  This has the advantage of letting
> us run it through NIR's optimizer which is pretty effective at shrinking
> things down.  In the common case of rgba8, the number of instructions
> emitted after NIR is done with it is half of what it was with the
> lowering happening in the back-end.  On the downside, the back-end's
> lowering is able to directly use predicates and the NIR lowering has to
> use IFs.
> 
> Shader-db results on Kaby Lake:
> 
>     total instructions in shared programs: 15166910 -> 15166872 (<.01%)
>     instructions in affected programs: 5895 -> 5857 (-0.64%)
>     helped: 15
>     HURT: 0
> 
> Clearly, we don't have that much image_load_store happening in the
> shaders in shader-db....

Looks like the untyped stride checks are back and the unnecessary typed
load checks (mistakenly copied from atomics) are gone.  Nice.

Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to