[PATCH] D51004: [llvm-strip] Allow only one input

Alexander Shaposhnikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 21 07:53:58 PDT 2018


alexshap added subscribers: MaskRay, paulsemel.
alexshap added a comment.

James, as for me, I think (though, don't insist) we might add a test for
it, but in this particular case it's not critical / long-term beneficial.
This is a temporary solution anyway. The thing is, that llvm-strip doesn't
support multiple inputs while binutils strip does.
So as soon as we fix it (add proper support for multiple input files), the
test would be gone / replaced with a meaningful test for multiple files.

Kind regards,
Alexander Shaposhnikov


Repository:
  rL LLVM

https://reviews.llvm.org/D51004





More information about the llvm-commits mailing list