diff options
| author | Richard Diamond <wichard@vitalitystudios.com> | 2015-08-05 22:10:57 +0000 |
|---|---|---|
| committer | Richard Diamond <wichard@vitalitystudios.com> | 2015-08-05 22:10:57 +0000 |
| commit | 559c1d72a94dcd7dba088438359422591aa3355b (patch) | |
| tree | bfe3f69fe6cc8654636f99258447ca558dccf77c /llvm | |
| parent | 36e7e52aa4f8d79c898d74f93711c4a0c78e253f (diff) | |
| download | bcm5719-llvm-559c1d72a94dcd7dba088438359422591aa3355b.tar.gz bcm5719-llvm-559c1d72a94dcd7dba088438359422591aa3355b.zip | |
Divide the primitive size in bits by eight so the initial load's alignment is in
bytes as expected. Tested with the included unit test.
llvm-svn: 244155
Diffstat (limited to 'llvm')
| -rw-r--r-- | llvm/lib/CodeGen/AtomicExpandPass.cpp | 2 | ||||
| -rw-r--r-- | llvm/test/Transforms/AtomicExpand/expand-atomic-rmw-initial-load.ll | 8 |
2 files changed, 9 insertions, 1 deletions
diff --git a/llvm/lib/CodeGen/AtomicExpandPass.cpp b/llvm/lib/CodeGen/AtomicExpandPass.cpp index 03b0ff32f15..c10648c6e3b 100644 --- a/llvm/lib/CodeGen/AtomicExpandPass.cpp +++ b/llvm/lib/CodeGen/AtomicExpandPass.cpp @@ -550,7 +550,7 @@ bool llvm::expandAtomicRMWToCmpXchg(AtomicRMWInst *AI, Builder.SetInsertPoint(BB); LoadInst *InitLoaded = Builder.CreateLoad(Addr); // Atomics require at least natural alignment. - InitLoaded->setAlignment(AI->getType()->getPrimitiveSizeInBits()); + InitLoaded->setAlignment(AI->getType()->getPrimitiveSizeInBits() / 8); Builder.CreateBr(LoopBB); // Start the main loop block now that we've taken care of the preliminaries. diff --git a/llvm/test/Transforms/AtomicExpand/expand-atomic-rmw-initial-load.ll b/llvm/test/Transforms/AtomicExpand/expand-atomic-rmw-initial-load.ll new file mode 100644 index 00000000000..197250ef455 --- /dev/null +++ b/llvm/test/Transforms/AtomicExpand/expand-atomic-rmw-initial-load.ll @@ -0,0 +1,8 @@ +; RUN: opt -S %s -atomic-expand -mtriple=i686-linux-gnu | FileCheck %s + +define i8 @test_initial_load(i8* %ptr, i8 %value) { + %res = atomicrmw nand i8* %ptr, i8 %value seq_cst + ret i8 %res +} +; CHECK-LABEL: @test_initial_load +; CHECK-NEXT: %1 = load i8, i8* %ptr, align 1 |
