[PATCH] D69146: [install-name-tool] Add first bits for install-name-tool

Alexander Shaposhnikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 29 02:41:38 PDT 2019


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


================
Comment at: llvm/tools/llvm-objcopy/CopyConfig.cpp:766
+    return createStringError(errc::invalid_argument,
+                             "llvm-install-name-tool expects a single input file");
+  Config.InputFilename = Positional[0];
----------------
jhenderson wrote:
> This line still doesn't look to be clang-formatted.
sorry, I changed it a bit after formatting, will update the patch, thanks for pointing out.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D69146





More information about the llvm-commits mailing list