[PATCH] D141024: [lld-macho][test] Simplify classrefs test

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 5 09:53:22 PST 2023


thakis accepted this revision.
thakis added a comment.
This revision is now accepted and ready to land.

lg



================
Comment at: lld/test/MachO/objc-classrefs-dedup.s:8
 # RUN: %lld -lSystem -dylib --icf=all %t/refs1.o %t/refs2.o %t/libdefs.dylib -o %t/out
-# RUN: llvm-objdump --macho --section-headers --bind %t/out | FileCheck %s \
-# RUN:   --implicit-check-not __objc_classrefs
-
-## Check that we only have 3 (unique) entries
-# CHECK:      Sections:
-# CHECK-NEXT: Idx Name             Size
-# CHECK:          __objc_classrefs 00000018
-
-## And only two binds
-# CHECK:       Bind table:
-# CHECK-NEXT:  segment  section           address  type     addend dylib    symbol
-# CHECK-DAG:   __DATA   __objc_classrefs  {{.*}}   pointer       0 libdefs  _OBJC_CLASS_$_Bar
-# CHECK-DAG:   __DATA   __objc_classrefs  {{.*}}   pointer       0 libdefs  _OBJC_CLASS_$_Foo
+# RUN: llvm-objdump --macho --objc-meta-data %t/out | FileCheck %s
+
----------------
nit: consider `llvm-otool -o` instead (does the same thing, just shorter)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D141024/new/

https://reviews.llvm.org/D141024



More information about the llvm-commits mailing list