[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