[PATCH] D106891: [AMDGPU] [Remarks] Emit optimization remarks for atomics generating CAS loop

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added inline comments.


================
Comment at: llvm/lib/CodeGen/AtomicExpandPass.cpp:618
       expandAtomicRMWToCmpXchg(AI, createCmpXchgInstFun);
+      Ctx.getSyncScopeNames(SSNs);
+      auto MemScope = SSNs[AI->getSyncScopeID()].empty()
----------------
Only if SSNs.empty().


================
Comment at: llvm/lib/CodeGen/AtomicExpandPass.cpp:624
+        Remark << "A compare and swap loop was generated for an "
+               << AI->getOpcodeName() << "operation at " << MemScope
+               << " memory scope";
----------------
I believe getOpcodeName() will return "atomicrmw" instead of the operation. Also missing space after it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106891