[PATCH] D53332: Access ADDLIB in llvm-ar via command line

Owen Reynolds via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 18 05:40:52 PDT 2018


gbreynoo added inline comments.


================
Comment at: tools/llvm-ar/llvm-ar.cpp:553
+    NMOrErr->MemberName = sys::path::filename(NMOrErr->MemberName);
+    Members.push_back(std::move(*NMOrErr));
+  }
----------------
mstorsjo wrote:
> Any reason why this path can't just call the default `addMember`?
It would mean repeating the getFile() call, I wasn't sure it was worth the overhead.


Repository:
  rL LLVM

https://reviews.llvm.org/D53332





More information about the llvm-commits mailing list