[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