aco: insert s_nop before VGPR deallocation
authorRhys Perry <pendingchaos02@gmail.com>
Tue, 25 Jul 2023 16:51:24 +0000 (17:51 +0100)
committerMarge Bot <emma+marge@anholt.net>
Wed, 26 Jul 2023 13:58:22 +0000 (13:58 +0000)
commit37fbfa655a4dd73a1c95ade29362d8e93f5d1fcc
treed0152656cc967f6499663f90d9a4acfb14a09197
parent7b29ae557ab9e86d75952c580afd87b6cb985373
aco: insert s_nop before VGPR deallocation

A hazard requires this, apparently.

https://reviews.llvm.org/D155681

fossil-db (gfx1100):
Totals from 23175 (17.36% of 133461) affected shaders:
Instrs: 41240100 -> 41263275 (+0.06%)
CodeSize: 211858524 -> 211951224 (+0.04%)
Latency: 469738570 -> 469738576 (+0.00%)
InvThroughput: 58013998 -> 58013999 (+0.00%)

Signed-off-by: Rhys Perry <pendingchaos02@gmail.com>
Reviewed-by: Daniel Schürmann <daniel@schuermann.dev>
Cc: mesa-stable
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/24320>
src/amd/compiler/aco_ir.cpp