no atomic NumSchedGroups

ro-i · ro-i · commit 4d53d16f5384 · 2025-05-01T13:43:51.000-05:00
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUIGroupLP.cpp b/llvm/lib/Target/AMDGPU/AMDGPUIGroupLP.cpp
@@ -229,13 +229,13 @@ class SchedGroup {
   SchedGroup(SchedGroupMask SGMask, std::optional<unsigned> MaxSize,
              ScheduleDAGInstrs *DAG, const SIInstrInfo *TII)
       : SGMask(SGMask), MaxSize(MaxSize), DAG(DAG), TII(TII) {
-    SGID = __atomic_fetch_add(&NumSchedGroups, 1, __ATOMIC_SEQ_CST);
+    SGID = NumSchedGroups++;
   }
 
   SchedGroup(SchedGroupMask SGMask, std::optional<unsigned> MaxSize, int SyncID,
              ScheduleDAGInstrs *DAG, const SIInstrInfo *TII)
       : SGMask(SGMask), MaxSize(MaxSize), SyncID(SyncID), DAG(DAG), TII(TII) {
-    SGID = __atomic_fetch_add(&NumSchedGroups, 1, __ATOMIC_SEQ_CST);
+    SGID = NumSchedGroups++;
   }
 };
 

Original file line number	Diff line number	Diff line change
`@@ -229,13 +229,13 @@ class SchedGroup {`
`229`	`229`	`SchedGroup(SchedGroupMask SGMask, std::optional<unsigned> MaxSize,`
`230`	`230`	`ScheduleDAGInstrs DAG, const SIInstrInfo TII)`
`231`	`231`	`: SGMask(SGMask), MaxSize(MaxSize), DAG(DAG), TII(TII) {`
`232`		`- SGID = __atomic_fetch_add(&NumSchedGroups, 1, __ATOMIC_SEQ_CST);`
	`232`	`+ SGID = NumSchedGroups++;`
`233`	`233`	`}`
`234`	`234`
`235`	`235`	`SchedGroup(SchedGroupMask SGMask, std::optional<unsigned> MaxSize, int SyncID,`
`236`	`236`	`ScheduleDAGInstrs DAG, const SIInstrInfo TII)`
`237`	`237`	`: SGMask(SGMask), MaxSize(MaxSize), SyncID(SyncID), DAG(DAG), TII(TII) {`
`238`		`- SGID = __atomic_fetch_add(&NumSchedGroups, 1, __ATOMIC_SEQ_CST);`
	`238`	`+ SGID = NumSchedGroups++;`
`239`	`239`	`}`
`240`	`240`	`};`
`241`	`241`