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

Unified Diff: tests_lit/assembler/arm32/vldr-vector.ll

Issue 1663053008: Fix vector load/stores in the ARM assembler. (Closed) Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Patch Set: Change crosstests to use filetype=obj. Created 4 years, 10 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
Index: tests_lit/assembler/arm32/vldr-vector.ll
diff --git a/tests_lit/assembler/arm32/vldr-vector.ll b/tests_lit/assembler/arm32/vldr-vector.ll
index 4078eb526e81ff2b18cc9746765444abf453a25e..3169bd686484857839c3ab6a4c89d8557201d470 100644
--- a/tests_lit/assembler/arm32/vldr-vector.ll
+++ b/tests_lit/assembler/arm32/vldr-vector.ll
@@ -1,73 +1,84 @@
; Show that we know how to translate vector load instructions.
+; Note: Uses -O2 to remove unnecessary loads/stores, resulting in only
+; one VLD1 instruction per function.
+
; REQUIRES: allow_dump
; Compile using standalone assembler.
; RUN: %p2i --filetype=asm -i %s --target=arm32 --args -O2 \
+; RUN: -reg-use=q11,r5 \
; RUN: | FileCheck %s --check-prefix=ASM
; Show bytes in assembled standalone code.
; RUN: %p2i --filetype=asm -i %s --target=arm32 --assemble --disassemble \
; RUN: --args -O2 \
+; RUN: -reg-use=q11,r5 \
; RUN: | FileCheck %s --check-prefix=DIS
; Compile using integrated assembler.
; RUN: %p2i --filetype=iasm -i %s --target=arm32 --args -O2 \
+; RUN: -reg-use=q11,r5 \
; RUN: | FileCheck %s --check-prefix=IASM
; Show bytes in assembled integrated code.
; RUN: %p2i --filetype=iasm -i %s --target=arm32 --assemble --disassemble \
; RUN: --args -O2 \
+; RUN: -reg-use=q11,r5 \
; RUN: | FileCheck %s --check-prefix=DIS
define internal <4 x float> @testDerefFloat4(<4 x float> *%p) {
; ASM-LABEL: testDerefFloat4:
-; DIS-LABEL: 00000000 <testDerefFloat4>:
+; DIS-LABEL: {{.+}} <testDerefFloat4>:
; IASM-LABEL: testDerefFloat4:
entry:
%ret = load <4 x float>, <4 x float>* %p, align 4
-; ASM: vld1.64 q0, [r0]
-; DIS: 0: f4200acf
+; ASM: vld1.32 q11, [r5]
+; DIS: {{.*}}: f4656a8f
+; IASM-NOT: vld1.32
ret <4 x float> %ret
}
define internal <4 x i32> @testDeref4i32(<4 x i32> *%p) {
; ASM-LABEL: testDeref4i32:
-; DIS-LABEL: 00000010 <testDeref4i32>:
+; DIS-LABEL: {{.+}} <testDeref4i32>:
; IASM-LABEL: testDeref4i32:
entry:
%ret = load <4 x i32>, <4 x i32>* %p, align 4
-; ASM: vld1.64 q0, [r0]
-; DIS: 10: f4200acf
+; ASM: vld1.32 q11, [r5]
+; DIS: {{.+}}: f4656a8f
+; IASM-NOT: vld1.32
ret <4 x i32> %ret
}
define internal <8 x i16> @testDeref8i16(<8 x i16> *%p) {
; ASM-LABEL: testDeref8i16:
-; DIS-LABEL: 00000020 <testDeref8i16>:
+; DIS-LABEL: {{.+}} <testDeref8i16>:
; IASM-LABEL: testDeref8i16:
entry:
%ret = load <8 x i16>, <8 x i16>* %p, align 2
-; ASM: vld1.64 q0, [r0]
-; DIS: 20: f4200acf
+; ASM: vld1.16 q11, [r5]
+; DIS: {{.+}}: f4656a4f
+; IASM-NOT: vld1.16
ret <8 x i16> %ret
}
define internal <16 x i8> @testDeref16i8(<16 x i8> *%p) {
; ASM-LABEL: testDeref16i8:
-; DIS-LABEL: 00000030 <testDeref16i8>:
+; DIS-LABEL: {{.+}} <testDeref16i8>:
; IASM-LABEL: testDeref16i8:
entry:
%ret = load <16 x i8>, <16 x i8>* %p, align 1
-; ASM: vld1.64 q0, [r0]
-; DIS: 30: f4200acf
+; ASM: vld1.8 q11, [r5]
+; DIS: {{.+}}: f4656a0f
+; IASM-NOT: vld1.8
ret <16 x i8> %ret
}

Powered by Google App Engine
This is Rietveld 408576698