[PATCH] D139959: [lld-macho][test] Add test case for section ordering

Jez Ng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 13 11:58:12 PST 2022


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

> This chooses to fake the sections vs creating them organically which is probably a bit easier to maintain.

Neat idea :D



================
Comment at: lld/test/MachO/section-order.s:8
 # RUN: %lld -dylib %t/2.o %t/1.o -o %t/21
+# RUN: %lld -dylib %t/2.o %t/1.o -o %t/22 \
+# RUN:     -add_empty_section __TEXT __objc_stubs \
----------------
`12` and `21` were so named to reflect the order of the input object files, so `22` doesn't exactly make sense here. Maybe name this `synth-section-order` instead?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139959



More information about the llvm-commits mailing list