[PATCH] D121366: Allow ImproveChain to get past relaxed atomics

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 23 12:47:12 PDT 2022


RKSimon added reviewers: RKSimon, craig.topper.
RKSimon added a comment.
Herald added a subscriber: StephenFan.

In D121366#3374969 <https://reviews.llvm.org/D121366#3374969>, @yoni-lavi wrote:

> I've confirmed my diff does not cause any test failures that aren't already present in the base revision (latest main).

Do you have tests that you can add that do show a change?



================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:24071
+      return {AN->isVolatile(), AN->isAtomic(),          AN->getBasePtr(),
+              Offset,           Optional<int64_t>(Size), AN->getMemOperand()};
+    }
----------------
is this how clang-format formats the code?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121366



More information about the llvm-commits mailing list