[llvm] [AMDGPU] Consider FLAT instructions for VMEM hazard detection (PR #137170)
Robert Imschweiler via llvm-commits
llvm-commits at lists.llvm.org
Sat Jun 14 12:22:47 PDT 2025
================
@@ -1417,9 +1417,8 @@ static bool shouldRunLdsBranchVmemWARHazardFixup(const MachineFunction &MF,
bool HasVmem = false;
for (auto &MBB : MF) {
for (auto &MI : MBB) {
- HasLds |= SIInstrInfo::isDS(MI);
- HasVmem |= (SIInstrInfo::isVMEM(MI) && !SIInstrInfo::isFLAT(MI)) ||
- SIInstrInfo::isSegmentSpecificFLAT(MI);
+ HasLds |= SIInstrInfo::isDS(MI) || SIInstrInfo::isLDSDMA(MI);
+ HasVmem |= SIInstrInfo::isVMEM(MI) && !SIInstrInfo::isLDSDMA(MI);
----------------
ro-i wrote:
So what we actually want for `HasVmem` is just plain `SIInstrInfo::isVMEM(MI)`?
https://github.com/llvm/llvm-project/pull/137170
More information about the llvm-commits
mailing list