[llvm] r220494 - Make this test a bit stricter.

Rafael Espindola rafael.espindola at gmail.com
Thu Oct 23 11:44:07 PDT 2014


Author: rafael
Date: Thu Oct 23 13:44:07 2014
New Revision: 220494

URL: http://llvm.org/viewvc/llvm-project?rev=220494&view=rev
Log:
Make this test a bit stricter.

This now:
* Forces the linker to include the internal definition.
* Checks the full output.

Added:
    llvm/trunk/test/Linker/Inputs/2003-01-30-LinkerRename.ll
Modified:
    llvm/trunk/test/Linker/2003-01-30-LinkerRename.ll

Modified: llvm/trunk/test/Linker/2003-01-30-LinkerRename.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Linker/2003-01-30-LinkerRename.ll?rev=220494&r1=220493&r2=220494&view=diff
==============================================================================
--- llvm/trunk/test/Linker/2003-01-30-LinkerRename.ll (original)
+++ llvm/trunk/test/Linker/2003-01-30-LinkerRename.ll Thu Oct 23 13:44:07 2014
@@ -1,10 +1,17 @@
-; This fails because the linker renames the external symbol not the internal 
-; one...
-
-; RUN: echo "define internal i32 @foo() { ret i32 7 } " | llvm-as > %t.1.bc
+; RUN: llvm-as %S/Inputs/2003-01-30-LinkerRename.ll -o %t.1.bc
 ; RUN: llvm-as %s -o %t.2.bc
 ; RUN: llvm-link %t.1.bc %t.2.bc -S | FileCheck %s
-; CHECK: internal{{.*}}@foo{{[0-9]}}()
 
-define i32 @foo() { ret i32 0 }
+; CHECK: @bar = global i32 ()* @foo2
+
+; CHECK:      define internal i32 @foo2() {
+; CHECK-NEXT:   ret i32 7
+; CHECK-NEXT: }
+
+; CHECK:      define i32 @foo() {
+; CHECK-NEXT:   ret i32 0
+; CHECK-NEXT: }
 
+define i32 @foo() {
+  ret i32 0
+}

Added: llvm/trunk/test/Linker/Inputs/2003-01-30-LinkerRename.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Linker/Inputs/2003-01-30-LinkerRename.ll?rev=220494&view=auto
==============================================================================
--- llvm/trunk/test/Linker/Inputs/2003-01-30-LinkerRename.ll (added)
+++ llvm/trunk/test/Linker/Inputs/2003-01-30-LinkerRename.ll Thu Oct 23 13:44:07 2014
@@ -0,0 +1,4 @@
+ at bar = global i32()* @foo
+define internal i32 @foo() {
+  ret i32 7
+}





More information about the llvm-commits mailing list