[clang-tools-extra] [clang] [llvm] [X86] Use plain load/store instead of cmpxchg16b for atomics with AVX (PR #74275)
Philip Reames via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 14 12:21:07 PST 2023
================
@@ -30115,12 +30126,16 @@ X86TargetLowering::shouldExpandAtomicLoadInIR(LoadInst *LI) const {
// If this a 64 bit atomic load on a 32-bit target and SSE2 is enabled, we
// can use movq to do the load. If we have X87 we can load into an 80-bit
// X87 register and store it to a stack temporary.
----------------
preames wrote:
Can you move the comment down to the case it applies to (inside the outer if). Currently, it binds to both cases which is misleading.
https://github.com/llvm/llvm-project/pull/74275
More information about the llvm-commits
mailing list