Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(416)

Unified Diff: test/Transforms/NaCl/atomic/lock_.ll

Issue 791053006: Add support for acquire, release, and acq_rel memory ordering in PNaCl (Closed) Base URL: https://chromium.googlesource.com/native_client/pnacl-llvm.git@master
Patch Set: Nits. Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « test/Transforms/NaCl/atomic/atomic_others.ll ('k') | test/Transforms/NaCl/resolve-pnacl-intrinsics.ll » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/Transforms/NaCl/atomic/lock_.ll
diff --git a/test/Transforms/NaCl/atomic/lock_.ll b/test/Transforms/NaCl/atomic/lock_.ll
index 44e43719719b45b706cac549697c3d85985df799..6f66f6706122c9db498ce85d6d2ad8c982f06231 100644
--- a/test/Transforms/NaCl/atomic/lock_.ll
+++ b/test/Transforms/NaCl/atomic/lock_.ll
@@ -20,8 +20,7 @@ define zeroext i8 @test_lock_test_and_set_i8(i8* %ptr, i8 zeroext %value) {
; CHECK-LABEL: @test_lock_release_i8
define void @test_lock_release_i8(i8* %ptr) {
- ; Note that the 'release' was changed to a 'seq_cst'.
- ; CHECK-NEXT: call void @llvm.nacl.atomic.store.i8(i8 0, i8* %ptr, i32 6)
+ ; CHECK-NEXT: call void @llvm.nacl.atomic.store.i8(i8 0, i8* %ptr, i32 4)
store atomic i8 0, i8* %ptr release, align 1
ret void ; CHECK-NEXT: ret void
}
@@ -35,8 +34,7 @@ define zeroext i16 @test_lock_test_and_set_i16(i16* %ptr, i16 zeroext %value) {
; CHECK-LABEL: @test_lock_release_i16
define void @test_lock_release_i16(i16* %ptr) {
- ; Note that the 'release' was changed to a 'seq_cst'.
- ; CHECK-NEXT: call void @llvm.nacl.atomic.store.i16(i16 0, i16* %ptr, i32 6)
+ ; CHECK-NEXT: call void @llvm.nacl.atomic.store.i16(i16 0, i16* %ptr, i32 4)
store atomic i16 0, i16* %ptr release, align 2
ret void ; CHECK-NEXT: ret void
}
@@ -50,8 +48,7 @@ define i32 @test_lock_test_and_set_i32(i32* %ptr, i32 %value) {
; CHECK-LABEL: @test_lock_release_i32
define void @test_lock_release_i32(i32* %ptr) {
- ; Note that the 'release' was changed to a 'seq_cst'.
- ; CHECK-NEXT: call void @llvm.nacl.atomic.store.i32(i32 0, i32* %ptr, i32 6)
+ ; CHECK-NEXT: call void @llvm.nacl.atomic.store.i32(i32 0, i32* %ptr, i32 4)
store atomic i32 0, i32* %ptr release, align 4
ret void ; CHECK-NEXT: ret void
}
@@ -65,8 +62,7 @@ define i64 @test_lock_test_and_set_i64(i64* %ptr, i64 %value) {
; CHECK-LABEL: @test_lock_release_i64
define void @test_lock_release_i64(i64* %ptr) {
- ; Note that the 'release' was changed to a 'seq_cst'.
- ; CHECK-NEXT: call void @llvm.nacl.atomic.store.i64(i64 0, i64* %ptr, i32 6)
+ ; CHECK-NEXT: call void @llvm.nacl.atomic.store.i64(i64 0, i64* %ptr, i32 4)
store atomic i64 0, i64* %ptr release, align 8
ret void ; CHECK-NEXT: ret void
}
« no previous file with comments | « test/Transforms/NaCl/atomic/atomic_others.ll ('k') | test/Transforms/NaCl/resolve-pnacl-intrinsics.ll » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698