[PATCH] D122664: [AMDGPU] Fix last remaining checks in perfhint.ll

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 29 11:31:34 PDT 2022


rampitec accepted this revision.
rampitec added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/test/CodeGen/AMDGPU/perfhint.ll:80
+; GCN: MemoryBound: 0
+; GCN: WaveLimiterHint : 1
 define amdgpu_kernel void @test_indirect_through_phi(float addrspace(1)* %arg) {
----------------
foad wrote:
> @rampitec do you want to try to come up with a test that really does result in "WaveLimiterHint : 0"? It seems hard because the default value for -amdgpu-indirect-access-weight is so high. (See also D83675.)
Do you mind to place FIXME here?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122664/new/

https://reviews.llvm.org/D122664



More information about the llvm-commits mailing list