[PATCH] D51660: [llvm-strip] Support stripping multiple input files
Jake Ehrlich via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 4 20:11:32 PDT 2018
jakehehrlich accepted this revision.
jakehehrlich added a comment.
Whoops...I meant to say the following already. I was confused why it was missing my LGTM.
Below no longer relevant:
I like the change, I'm quite satisfied with the coverage given by your test. You're welcome to optimize the test a bit as Alex has outlined (I would be satisfied with that testing strategy as well).
Repository:
rL LLVM
https://reviews.llvm.org/D51660
More information about the llvm-commits
mailing list