[lld] d2b845d - [lld-macho] Disable invalid/stub-link.s test for Mac

Jez Ng via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 27 11:24:36 PDT 2020


Author: Jez Ng
Date: 2020-08-27T11:24:22-07:00
New Revision: d2b845dd6c90508c6258b4023fdf9ab5519fee3f

URL: https://github.com/llvm/llvm-project/commit/d2b845dd6c90508c6258b4023fdf9ab5519fee3f
DIFF: https://github.com/llvm/llvm-project/commit/d2b845dd6c90508c6258b4023fdf9ab5519fee3f.diff

LOG: [lld-macho] Disable invalid/stub-link.s test for Mac

It seems to be failing on some Google Buildbots.

This diff also includes a minor fix for the install name of one of
libSystem's re-exports. I don't think it's the cause of the test
failure, though. The wrong install name just meant that the symbol
lookup failure would still happen, but it would have been caused by the
re-export not being found, instead of the arch failing to match.

Differential Revision: https://reviews.llvm.org/D86728

Added: 
    

Modified: 
    lld/test/MachO/Inputs/iPhoneSimulator.sdk/usr/lib/libSystem.tbd
    lld/test/MachO/invalid/stub-link.s

Removed: 
    


################################################################################
diff  --git a/lld/test/MachO/Inputs/iPhoneSimulator.sdk/usr/lib/libSystem.tbd b/lld/test/MachO/Inputs/iPhoneSimulator.sdk/usr/lib/libSystem.tbd
index 56246f480d99..86d576b507cd 100644
--- a/lld/test/MachO/Inputs/iPhoneSimulator.sdk/usr/lib/libSystem.tbd
+++ b/lld/test/MachO/Inputs/iPhoneSimulator.sdk/usr/lib/libSystem.tbd
@@ -12,7 +12,7 @@ exports:
 archs:            [ i386, x86_64 ]
 uuids:            [ 'i386: 00000000-0000-0000-0000-000000000002', 'x86_64: 00000000-0000-0000-0000-000000000003' ]
 platform:         ios
-install-name:     '/usr/lib/libcache.dylib'
+install-name:     '/usr/lib/system/libcache.dylib'
 current-version:  83
 parent-umbrella:  System
 exports:

diff  --git a/lld/test/MachO/invalid/stub-link.s b/lld/test/MachO/invalid/stub-link.s
index 8160ff69e071..bcbdea1220a4 100644
--- a/lld/test/MachO/invalid/stub-link.s
+++ b/lld/test/MachO/invalid/stub-link.s
@@ -1,3 +1,6 @@
+## FIXME: This test seems to be failing on some Google Mac buildbots for
+## unclear reasons, so it's disabled for now. See D85404 for details.
+# UNSUPPORTED: darwin
 # REQUIRES: x86
 
 # RUN: mkdir -p %t


        


More information about the llvm-commits mailing list