[llvm] r179978 - ARM: fix part of test which actually needed an asserts build

Tim Northover Tim.Northover at arm.com
Sun Apr 21 05:20:19 PDT 2013


Author: tnorthover
Date: Sun Apr 21 07:20:19 2013
New Revision: 179978

URL: http://llvm.org/viewvc/llvm-project?rev=179978&view=rev
Log:
ARM: fix part of test which actually needed an asserts build

This should fix a buildbot failure that occurred after r179977.

Added:
    llvm/trunk/test/CodeGen/ARM/gpr-paired-spill-thumbinst.ll
Modified:
    llvm/trunk/test/CodeGen/ARM/gpr-paired-spill.ll

Added: llvm/trunk/test/CodeGen/ARM/gpr-paired-spill-thumbinst.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/ARM/gpr-paired-spill-thumbinst.ll?rev=179978&view=auto
==============================================================================
--- llvm/trunk/test/CodeGen/ARM/gpr-paired-spill-thumbinst.ll (added)
+++ llvm/trunk/test/CodeGen/ARM/gpr-paired-spill-thumbinst.ll Sun Apr 21 07:20:19 2013
@@ -0,0 +1,30 @@
+; REQUIRES: asserts
+; RUN: llc -mtriple=thumbv7-none-linux-gnueabi -debug -o /dev/null < %s 2>&1 | FileCheck %s
+
+; This test makes sure spills of 64-bit pairs in Thumb mode actually
+; generate thumb instructions. Previously we were inserting an ARM
+; STMIA which happened to have the same encoding.
+
+define void @foo(i64* %addr) {
+  %val1 = tail call i64 asm sideeffect "ldrexd $0, ${0:H}, [r0]", "=&r,r"(i64* %addr)
+  %val2 = tail call i64 asm sideeffect "ldrexd $0, ${0:H}, [r0]", "=&r,r"(i64* %addr)
+  %val3 = tail call i64 asm sideeffect "ldrexd $0, ${0:H}, [r0]", "=&r,r"(i64* %addr)
+  %val4 = tail call i64 asm sideeffect "ldrexd $0, ${0:H}, [r0]", "=&r,r"(i64* %addr)
+  %val5 = tail call i64 asm sideeffect "ldrexd $0, ${0:H}, [r0]", "=&r,r"(i64* %addr)
+  %val6 = tail call i64 asm sideeffect "ldrexd $0, ${0:H}, [r0]", "=&r,r"(i64* %addr)
+  %val7 = tail call i64 asm sideeffect "ldrexd $0, ${0:H}, [r0]", "=&r,r"(i64* %addr)
+
+  ; Make sure we are actually creating the Thumb versions of the spill
+  ; instructions.
+; CHECK: t2STRDi8
+; CHECK: t2LDRDi8
+
+  store volatile i64 %val1, i64* %addr
+  store volatile i64 %val2, i64* %addr
+  store volatile i64 %val3, i64* %addr
+  store volatile i64 %val4, i64* %addr
+  store volatile i64 %val5, i64* %addr
+  store volatile i64 %val6, i64* %addr
+  store volatile i64 %val7, i64* %addr
+  ret void
+}

Modified: llvm/trunk/test/CodeGen/ARM/gpr-paired-spill.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/ARM/gpr-paired-spill.ll?rev=179978&r1=179977&r2=179978&view=diff
==============================================================================
--- llvm/trunk/test/CodeGen/ARM/gpr-paired-spill.ll (original)
+++ llvm/trunk/test/CodeGen/ARM/gpr-paired-spill.ll Sun Apr 21 07:20:19 2013
@@ -1,7 +1,6 @@
 ; RUN: llc -mtriple=armv7-none-linux-gnueabi -verify-machineinstrs < %s | FileCheck %s --check-prefix=CHECK-WITH-LDRD
 ; RUN: llc -mtriple=armv4-none-linux-gnueabi -verify-machineinstrs < %s | FileCheck %s --check-prefix=CHECK-WITHOUT-LDRD
 ; RUN: llc -mtriple=thumbv7-none-linux-gnueabi -verify-machineinstrs < %s | FileCheck %s --check-prefix=CHECK-WITH-LDRD
-; RUN: llc -mtriple=thumbv7-none-linux-gnueabi -debug -o /dev/null < %s 2>&1 | FileCheck %s --check-prefix=INSTRS-ARE-THUMB
 
 define void @foo(i64* %addr) {
   %val1 = tail call i64 asm sideeffect "ldrexd $0, ${0:H}, [r0]", "=&r,r"(i64* %addr)
@@ -34,11 +33,6 @@ define void @foo(i64* %addr) {
 ; CHECK-WITHOUT-LDRD: ldm [[ADDRREG]], {r{{[0-9]+}}, r{{[0-9]+}}}
 ; CHECK-WITHOUT-LDRD: ldm sp, {r{{[0-9]+}}, r{{[0-9]+}}}
 
-  ; Make sure we are actually creating the Thumb versions of the spill
-  ; instructions.
-; INSTRS-ARE-THUMB: t2STRDi8
-; INSTRS-ARE-THUMB: t2LDRDi8
-
   store volatile i64 %val1, i64* %addr
   store volatile i64 %val2, i64* %addr
   store volatile i64 %val3, i64* %addr





More information about the llvm-commits mailing list