Skip to content

aco/gfx11+: don't use VOP3 v_swap_b16

Georg Lehmann requested to merge DadSchoorse/mesa:aco-no-vop3-swap_b16 into main

What does this MR do and why?

aco/gfx11+: don't use VOP3 v_swap_b16

v_swap_b16 is not offically supported as VOP3, so it can't be used with v128-255.
Tests show that VOP3 appears to work correctly, but according to AMD that should
not be relied on.

https://github.com/llvm/llvm-project/pull/100442#discussion_r1703929676

Foz-DB Navi31:
Totals from 6 (0.01% of 79395) affected shaders:
Instrs: 64799 -> 65932 (+1.75%)
CodeSize: 360180 -> 368440 (+2.29%)
Latency: 1364648 -> 1365922 (+0.09%)
InvThroughput: 635843 -> 636475 (+0.10%)
Copies: 14766 -> 15698 (+6.31%)
VALU: 38743 -> 39675 (+2.41%)

Fixes: 80b8bbf0c5c ("aco/gfx11: use v_swap_b16")

Merge request reports