[PATCH] D88674: [install-name-tool] Add --delete_all_rpaths to llvm-install-name-tool
Alexander Shaposhnikov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 7 16:47:29 PDT 2020
alexshap added a comment.
@rupprecht - in general, I'm not against using regex, however, as I mentioned above, it seems like it's not a very frequent use case + there are not so many rpaths (this is quite different from matching symbol names or section names).
Adding more options (--regex) + the required plumbing + tests coverage has its price, honestly I'm not convinced that it's strictly necessary in this particular case (when e.g. a binary has 2 or 3 rpaths).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D88674/new/
https://reviews.llvm.org/D88674
More information about the llvm-commits
mailing list