[PATCH] D106729: [AMDGPU] Support non-strictly stronger memory orderings in SIMemoryLegalizer

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 26 06:56:21 PDT 2021


foad added inline comments.


================
Comment at: llvm/include/llvm/Support/AtomicOrdering.h:140-141
+                                              AtomicOrdering Other) {
+  if (((AO == AtomicOrdering::Acquire) && (Other == AtomicOrdering::Release)) ||
+      ((AO == AtomicOrdering::Release) && (Other == AtomicOrdering::Acquire)))
+    return AtomicOrdering::AcquireRelease;
----------------
You don't need any of these innermost parentheses.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106729



More information about the llvm-commits mailing list