[PATCH] D81527: [llvm-install-name-tool] Add `delete_rpath` option

Sameer Arora via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 11 09:20:08 PDT 2020


sameerarora101 updated this revision to Diff 270162.
sameerarora101 marked 2 inline comments as done.
sameerarora101 added a comment.

Updates:

- Change `Config.RPathsToRemove` to a set.
- No error for duplicate rpaths
- incorporating above inline comments for test file


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D81527/new/

https://reviews.llvm.org/D81527

Files:
  llvm/test/tools/llvm-objcopy/MachO/install-name-tool-delete-rpath.test
  llvm/tools/llvm-objcopy/CopyConfig.cpp
  llvm/tools/llvm-objcopy/CopyConfig.h
  llvm/tools/llvm-objcopy/InstallNameToolOpts.td
  llvm/tools/llvm-objcopy/MachO/MachOObjcopy.cpp
  llvm/tools/llvm-objcopy/MachO/Object.cpp
  llvm/tools/llvm-objcopy/MachO/Object.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D81527.270162.patch
Type: text/x-patch
Size: 8607 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200611/ba8a7a73/attachment.bin>


More information about the llvm-commits mailing list