[PATCH] D82051: [llvm-install-name-tool] Add -rpath option

[Sameer Arora via Phabricator via llvm-commits]

sameerarora101 marked an inline comment as done.
sameerarora101 added inline comments.


================
Comment at: llvm/tools/llvm-objcopy/CopyConfig.cpp:853
+
+    // Cannot specify duplicate -rpath entries
+    auto It1 = std::find_if(
----------------
sameerarora101 wrote:
> Unlike `-delete_rpath`, we thought it was necessary to check duplication in case of `-rpath`. This is because it would not be useful to do something like `-rpath A B .... -rpath A C` or something like `-rpath A B ... -rpath B C`. Additionally,  this also matches XCode's -rpath behavior.
I can remove this check if people feel the need?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82051