[PATCH] D97392: [amdgpu] Atomic should be source of divergence.

Michael Liao via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 24 12:06:17 PST 2021


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


================
Comment at: llvm/lib/Target/AMDGPU/SIISelLowering.cpp:11808
+  case ISD::ATOMIC_LOAD_AND:
+  case ISD::ATOMIC_LOAD_CLR:
+  case ISD::ATOMIC_LOAD_OR:
----------------
arsenm wrote:
> I didn't know about this one
Only AArch64 uses that. Just add that conservatively.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97392



More information about the llvm-commits mailing list