[PATCH] D38332: [llvm-objcopy] Add ability to remove multiple sections by name

[James Henderson via Phabricator via llvm-commits]

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

LGTM with the comment change, and obviously once https://reviews.llvm.org/D38260 lands.



================
Comment at: tools/llvm-objcopy/llvm-objcopy.cpp:59
                                "\n\tbinary"));
 // TODO: make this a cl::list to support removing multiple sections
+cl::list<std::string> ToRemove("remove-section",
----------------
You can remove the TODO comment now!


Repository:
  rL LLVM

https://reviews.llvm.org/D38332