[PATCH] D96639: [AMDGPU] Add two TSFlags: IsAtomicNoRtn and IsAtomicRtn
Jay Foad via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 15 11:17:56 PST 2021
foad accepted this revision.
foad added a comment.
LGTM, thanks.
================
Comment at: llvm/lib/Target/AMDGPU/SIFormMemoryClauses.cpp:115
return false;
- if (AMDGPU::getAtomicNoRetOp(MI.getOpcode()) != -1 ||
- AMDGPU::getAtomicRetOp(MI.getOpcode()) != -1)
+ if (SIInstrInfo::isAtomic(MI))
return false;
----------------
I think this is dead too since it is covered by the mayStore just above.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96639/new/
https://reviews.llvm.org/D96639
More information about the llvm-commits
mailing list