[PATCH] D153007: [AMDGPU] Switch to the new cl option amdgpu-atomic-optimizer-strategy.

[Pravin Jagtap via Phabricator via llvm-commits]

pravinjagtap added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUAtomicOptimizer.cpp:867
             B.CreateIntrinsic(Intrinsic::amdgcn_strict_wwm, Ty, ExclScan);
-      } else {
+      } else if (ScanImpl == ScanOptions::Iterative) {
         LaneOffset = ExclScan;
----------------
arsenm wrote:
> So the pass behavior with none is to just crash?
Execution will not reach here for `None` strategy. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153007