[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