[PATCH] D101962: [AMDGPU] Switch AnnotateUniformValues to MemorySSA

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 5 18:37:13 PDT 2021


rampitec updated this revision to Diff 343263.
rampitec added a comment.

- SetVector include also not needed anymore.
- MSSAWalker is not always needed and cached by the MSSA anyway, so request it only when needed.

PSDB passed, I am about to submit it after check-llvm.


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

https://reviews.llvm.org/D101962

Files:
  llvm/lib/Target/AMDGPU/AMDGPUAnnotateUniformValues.cpp
  llvm/test/CodeGen/AMDGPU/coalescer_remat.ll
  llvm/test/CodeGen/AMDGPU/global-smrd-unknown.ll
  llvm/test/CodeGen/AMDGPU/llc-pipeline.ll
  llvm/test/CodeGen/AMDGPU/soft-clause-exceeds-register-budget.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D101962.343263.patch
Type: text/x-patch
Size: 10952 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210506/553b6c0f/attachment-0001.bin>


More information about the llvm-commits mailing list