[PATCH] D38335: [llvm-objcopy] Add support for --strip-sections to remove all section headers leaving only program headers and loadable segment data
James Henderson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 9 01:55:10 PDT 2017
jhenderson accepted this revision.
jhenderson added a comment.
This revision is now accepted and ready to land.
LGTM.
================
Comment at: test/tools/llvm-objcopy/strip-sections.test:28
+#CHECK-NEXT: Ident {
+#CHECK-NEXT: Magic: (7F 45 4C 46)
+#CHECK-NEXT: Class: 64-bit (0x2)
----------------
FWIW, I don't think you need to check all the fields, just the section header ones, but if you think it's useful, I'm not opposed to it.
Repository:
rL LLVM
https://reviews.llvm.org/D38335
More information about the llvm-commits
mailing list