[PATCH] D101065: [lld/mac] slightly improve weak-private-extern.s test

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 22 07:00:59 PDT 2021


thakis created this revision.
thakis added a reviewer: lld-macho.
Herald added a project: lld-macho.
thakis requested review of this revision.

- __got is in --bind output, so print that too (makes the test a bit stronger)
- WEAK_DEFINES, BINDS_TO_WEAK are in the mach-o header, so --private-header is enough, no need for --all-headers (makes the test a bit easier to work with when it fails)


https://reviews.llvm.org/D101065

Files:
  lld/test/MachO/weak-private-extern.s


Index: lld/test/MachO/weak-private-extern.s
===================================================================
--- lld/test/MachO/weak-private-extern.s
+++ lld/test/MachO/weak-private-extern.s
@@ -3,11 +3,11 @@
 # RUN: llvm-mc -filetype=obj -triple=x86_64-apple-macos %s -o %t.o
 # RUN: %lld -dylib %t.o -o %t.dylib -lSystem
 
-# RUN: llvm-objdump --macho --weak-bind %t.dylib | FileCheck %s
+# RUN: llvm-objdump --macho --bind --weak-bind %t.dylib | FileCheck %s
 # CHECK-NOT: __got
 # CHECK-NOT: __la_symbol_ptr
 
-# RUN: llvm-objdump --macho --all-headers %t.dylib | \
+# RUN: llvm-objdump --macho --private-header %t.dylib | \
 # RUN:     FileCheck --check-prefix=HEADERS %s
 # HEADERS-NOT: WEAK_DEFINES
 # HEADERS-NOT: BINDS_TO_WEAK


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D101065.339613.patch
Type: text/x-patch
Size: 739 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210422/edc6a094/attachment.bin>


More information about the llvm-commits mailing list