test/Transforms/NaCl/remove-asm-memory.ll - Issue 1151093004: Changes from 3.7 merge to files not in upstream

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: test/Transforms/NaCl/remove-asm-memory.ll

Issue 1151093004: Changes from 3.7 merge to files not in upstream (Closed) Base URL: https://chromium.googlesource.com/native_client/pnacl-llvm.git@master

Patch Set: Created 5 years, 7 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

« test/Transforms/NaCl/internalize-used-globals.ll ('K') | « test/Transforms/NaCl/promote-integers.ll ('k') | test/Transforms/NaCl/replace-ptrs-with-ints.ll » ('j') | test/Transforms/NaCl/replace-ptrs-with-ints.ll » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: test/Transforms/NaCl/remove-asm-memory.ll

diff --git a/test/Transforms/NaCl/remove-asm-memory.ll b/test/Transforms/NaCl/remove-asm-memory.ll

index ae799a74597c8c152e3ddd5d50e63ec842552ea1..cd3f99c83e4132a66e6e71e912e8e447909f9650 100644

--- a/test/Transforms/NaCl/remove-asm-memory.ll

+++ b/test/Transforms/NaCl/remove-asm-memory.ll

@@ -39,31 +39,31 @@ define void @memory_assembly_encoding_test() {

define void @memory_assembly_ordering_test() {

; CHECK: @memory_assembly_ordering_test()

- %1 = load i32* @a, align 4

+ %1 = load i32, i32* @a, align 4

store i32 %1, i32* @b, align 4

call void asm sideeffect "", "~{memory}"()

fence seq_cst

call void asm sideeffect "", "~{memory}"()

- ; CHECK-NEXT: %1 = load i32* @a, align 4

+ ; CHECK-NEXT: %1 = load i32, i32* @a, align 4

; CHECK-NEXT: store i32 %1, i32* @b, align 4

; CHECK-NEXT: call void @llvm.nacl.atomic.fence.all()

; Redundant load from the previous location, and store to the same

; location (making the previous one dead). Shouldn't get eliminated

; because of the fence.

- %2 = load i32* @a, align 4

+ %2 = load i32, i32* @a, align 4

store i32 %2, i32* @b, align 4

call void asm sideeffect "", "~{memory}"()

fence seq_cst

call void asm sideeffect "", "~{memory}"()

- ; CHECK-NEXT: %2 = load i32* @a, align 4

+ ; CHECK-NEXT: %2 = load i32, i32* @a, align 4

; CHECK-NEXT: store i32 %2, i32* @b, align 4

; CHECK-NEXT: call void @llvm.nacl.atomic.fence.all()

; Same here.

- %3 = load i32* @a, align 4

+ %3 = load i32, i32* @a, align 4

store i32 %3, i32* @b, align 4

- ; CHECK-NEXT: %3 = load i32* @a, align 4

+ ; CHECK-NEXT: %3 = load i32, i32* @a, align 4

; CHECK-NEXT: store i32 %3, i32* @b, align 4

ret void

@@ -74,13 +74,13 @@ define void @memory_assembly_ordering_test() {

; stores should get eliminated.

define void @memory_ordering_test() {

; ELIM: @memory_ordering_test()

- %1 = load i32* @a, align 4

+ %1 = load i32, i32* @a, align 4

store i32 %1, i32* @b, align 4

- %2 = load i32* @a, align 4

+ %2 = load i32, i32* @a, align 4

store i32 %2, i32* @b, align 4

- %3 = load i32* @a, align 4

+ %3 = load i32, i32* @a, align 4

store i32 %3, i32* @b, align 4

- ; ELIM-NEXT: %1 = load i32* @a, align 4

+ ; ELIM-NEXT: %1 = load i32, i32* @a, align 4

; ELIM-NEXT: store i32 %1, i32* @b, align 4

ret void