[PATCH] D93585: [AArch64] Enable out-of-line atomics by default.
    Tim Northover via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Sun Dec 20 03:00:07 PST 2020
    
    
  
t.p.northover added a comment.
As it stands this needs to be a platform-level change. The compiler-rt stuff may now compile outside Linux (thanks!), but the capability detection is still fundamentally Linux (and even glibc) only so nothing else would use it as intended.
I'd be wary of making it universal even if that's fixed though, unless it turns out that the code-size improvement outweighs the extra operations.
Speaking of which, do you have any benchmark numbers?
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93585/new/
https://reviews.llvm.org/D93585
    
    
More information about the llvm-commits
mailing list