[PATCH] D42854: [AMDGPU] Suppress redundant waitcnt instrs

Mark Searles via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 6 17:57:39 PST 2018


msearles marked an inline comment as done.
msearles added inline comments.


================
Comment at: lib/Target/AMDGPU/SIInsertWaitcnts.cpp:1533
       // Leaving the previously existing waitcnts is conservatively correct.
-      if (CompilerGeneratedWaitcntSet.find(&Inst) ==
-          CompilerGeneratedWaitcntSet.end())
+      if (TrackedWaitcntSet.find(&Inst) ==
+          TrackedWaitcntSet.end())
----------------
rampitec wrote:
> !TrackedWaitcntSet.count(&Inst) I guess.
Done; made equivalent changes in several other places.


https://reviews.llvm.org/D42854





More information about the llvm-commits mailing list