[llvm] r267722 - Remove size 1 from check as that isn't part of what the test is meant to be testing.

Kristof Beyls via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 27 08:03:09 PDT 2016


Author: kbeyls
Date: Wed Apr 27 10:03:09 2016
New Revision: 267722

URL: http://llvm.org/viewvc/llvm-project?rev=267722&view=rev
Log:
Remove size 1 from check as that isn't part of what the test is meant to be testing.

This test also runs on e.g. ARM-native builds when the X86 backend is also
built.  This test produces code for the default instruction set, even though it
is in a "X86" sub-directory. Given that this test doesn't seem to be testing
anything architecture-specific, it seems it's best to adapt the check to not
check for an architecture-dependent value (the size of the function), rather
than hard-code the test to target x86.


Modified:
    llvm/trunk/test/tools/gold/X86/visibility.ll

Modified: llvm/trunk/test/tools/gold/X86/visibility.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/tools/gold/X86/visibility.ll?rev=267722&r1=267721&r2=267722&view=diff
==============================================================================
--- llvm/trunk/test/tools/gold/X86/visibility.ll (original)
+++ llvm/trunk/test/tools/gold/X86/visibility.ll Wed Apr 27 10:03:09 2016
@@ -9,7 +9,7 @@
 
 ; CHECK:      Name: foo
 ; CHECK-NEXT: Value:
-; CHECK-NEXT: Size: 1
+; CHECK-NEXT: Size:
 ; CHECK-NEXT: Binding: Global
 ; CHECK-NEXT: Type: Function
 ; CHECK-NEXT: Other [




More information about the llvm-commits mailing list