[PATCH] D100281: [AMDGPU] Revise handling of preexisting waitcnt

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIInsertWaitcnts.cpp:814
+       &*II != MI; II = NextI, ++NextI) {
+    if (II->isDebugInstr())
+      continue;
----------------
isMetaInstruction() probably?


================
Comment at: llvm/lib/Target/AMDGPU/SIInsertWaitcnts.cpp:1158
+  // Verify that the wait is actually needed.
+  ScoreBrackets.simplifyWaitcnt(Wait);
 
----------------
We probably still want to support forced waitcounts for debugging purposes. A debug trap for memory violation usually happens at a wait count, so it is easier to debug with explicit waits for each memory access.


================
Comment at: llvm/lib/Target/AMDGPU/SIInsertWaitcnts.cpp:1601
+
+  if (!MFI->isEntryFunction()) {
+    // Wait for any outstanding memory operations that the input registers may
----------------
Isn't it a separate change?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100281