[llvm] [X86] Use fence(seq_cst) in IdempotentRMWIntoFencedLoad (PR #126521)

Valentin Churavy via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 11 02:04:47 PST 2025


================
@@ -31807,21 +31807,10 @@ X86TargetLowering::lowerIdempotentRMWIntoFencedLoad(AtomicRMWInst *AI) const {
   // otherwise, we might be able to be more aggressive on relaxed idempotent
   // rmw. In practice, they do not look useful, so we don't try to be
   // especially clever.
-  if (SSID == SyncScope::SingleThread)
----------------
vchuravy wrote:

In  https://github.com/llvm/llvm-project/pull/126521/commits/02f6abd67801b008aa43891b8915a9b6b6d5c71c I added a test

and https://github.com/llvm/llvm-project/pull/126521/commits/a01668d4e65115e4e68bd1efe4920cdcc74724e9 shows the changes with this PR


https://github.com/llvm/llvm-project/pull/126521


More information about the llvm-commits mailing list