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.
rust-lang · Feb 4, 2016 · 1096e7a · 1096e7a
2 parents 3c9442f + 01112d1
commit 1096e7a
Showing 1 changed file with 0 additions and 2 deletions.
diff --git 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));