Auto merge of #30962 - Amanieu:non_volatile_atomic, r=alexcrichton

Rust currently emits atomic loads and stores with the LLVM `volatile` qualifier. This is unnecessary and prevents LLVM from performing optimization on these atomic operations.

1 files changed, 0 insertions, 2 deletions

diff --git a/src/rustllvm/RustWrapper.cpp b/src/rustllvm/RustWrapper.cpp
index 0c18051dac7..ecf9146a1d0 100644
--- a/src/rustllvm/RustWrapper.cpp
+++ b/src/rustllvm/RustWrapper.cpp
@@ -170,7 +170,6 @@ extern "C" LLVMValueRef LLVMBuildAtomicLoad(LLVMBuilderRef B,
                                             AtomicOrdering order,
                                             unsigned alignment) {
     LoadInst* li = new LoadInst(unwrap(source),0);
-    li->setVolatile(true);
     li->setAtomic(order);
     li->setAlignment(alignment);
     return wrap(unwrap(B)->Insert(li, Name));
@@ -182,7 +181,6 @@ extern "C" LLVMValueRef LLVMBuildAtomicStore(LLVMBuilderRef B,
                                              AtomicOrdering order,
                                              unsigned alignment) {
     StoreInst* si = new StoreInst(unwrap(val),unwrap(target));
-    si->setVolatile(true);
     si->setAtomic(order);
     si->setAlignment(alignment);
     return wrap(unwrap(B)->Insert(si));