[PATCH] D82812: [llvm-install-name-tool] Merge rpath with id/change

[Shoaib Meenai via Phabricator via llvm-commits]

smeenai accepted this revision.
smeenai added inline comments.


================
Comment at: llvm/tools/llvm-objcopy/MachO/MachOObjcopy.cpp:169-170
+  for (const auto &OldNew : Config.RPathsToUpdate) {
+    StringRef Old = OldNew.getFirst();
+    StringRef New = OldNew.getSecond();
+    if (RPaths.count(Old) == 0)
----------------
sameerarora101 wrote:
> this is the lates update. Would it work on Darwin? thanks
Yup, that builds. I believe this is a libc++ bug though, and I commented on https://bugs.llvm.org/show_bug.cgi?id=17550#c11


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82812