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

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


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

On Tue, Aug 21, 2018 at 2:02 AM, James Henderson via Phabricator <
reviews at reviews.llvm.org> wrote:

> jhenderson added a comment.
>
> Shouldn't this have a test?
>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D51004
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180821/bc6cbbf6/attachment.html>


More information about the llvm-commits mailing list