[PATCH] D65539: [llvm-objcopy][MachO] Implement a layout algorithm for executables

Seiya Nuta via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 6 04:13:06 PDT 2019


seiya added inline comments.


================
Comment at: llvm/test/tools/llvm-objcopy/MachO/basic-executable-copy.test:5
+
+--- !mach-o
+FileHeader:
----------------
jhenderson wrote:
> There's a lot going on in this YAML. Is it the minimal it can be to show all the behaviour, or can it be reduced further?
It's too long but I think we should keep this verbose YAML to test all executable/shared library-specific load commands like LC_LOAD_DYLINKER.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65539





More information about the llvm-commits mailing list