[llvm] [NFC][AMDGPU] Remove unnecessary member `ForceEmitZeroWaitcnts` (PR #112114)
via llvm-commits
llvm-commits at lists.llvm.org
Sat Oct 12 20:01:42 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-backend-amdgpu
Author: Shilei Tian (shiltian)
<details>
<summary>Changes</summary>
We can use `ForceEmitZeroFlag` directly.
---
Full diff: https://github.com/llvm/llvm-project/pull/112114.diff
1 Files Affected:
- (modified) llvm/lib/Target/AMDGPU/SIInsertWaitcnts.cpp (+1-5)
``````````diff
diff --git a/llvm/lib/Target/AMDGPU/SIInsertWaitcnts.cpp b/llvm/lib/Target/AMDGPU/SIInsertWaitcnts.cpp
index 15f4114826e401..9866ecbdddb608 100644
--- a/llvm/lib/Target/AMDGPU/SIInsertWaitcnts.cpp
+++ b/llvm/lib/Target/AMDGPU/SIInsertWaitcnts.cpp
@@ -611,9 +611,6 @@ class SIInsertWaitcnts : public MachineFunctionPass {
MapVector<MachineBasicBlock *, BlockInfo> BlockInfos;
- // ForceEmitZeroWaitcnts: force all waitcnts insts to be s_waitcnt 0
- // because of amdgpu-waitcnt-forcezero flag
- bool ForceEmitZeroWaitcnts;
bool ForceEmitWaitcnt[NUM_INST_CNTS];
// In any given run of this pass, WCG will point to one of these two
@@ -1828,7 +1825,7 @@ bool SIInsertWaitcnts::generateWaitcntInstBefore(MachineInstr &MI,
// Verify that the wait is actually needed.
ScoreBrackets.simplifyWaitcnt(Wait);
- if (ForceEmitZeroWaitcnts)
+ if (ForceEmitZeroFlag)
Wait = WCG->getAllZeroWaitcnt(/*IncludeVSCnt=*/false);
if (ForceEmitWaitcnt[LOAD_CNT])
@@ -2424,7 +2421,6 @@ bool SIInsertWaitcnts::runOnMachineFunction(MachineFunction &MF) {
WCG = &WCGPreGFX12;
}
- ForceEmitZeroWaitcnts = ForceEmitZeroFlag;
for (auto T : inst_counter_types())
ForceEmitWaitcnt[T] = false;
``````````
</details>
https://github.com/llvm/llvm-project/pull/112114
More information about the llvm-commits
mailing list