[PATCH] D69146: [install-name-tool] Add first bits for install-name-tool
Saleem Abdulrasool via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 29 14:07:01 PDT 2019
compnerd accepted this revision.
compnerd added inline comments.
This revision is now accepted and ready to land.
================
Comment at: llvm/tools/llvm-objcopy/llvm-objcopy.cpp:347
Expected<DriverConfig> DriverConfig =
- IsStrip ? parseStripOptions(Args, reportWarning)
- : parseObjcopyOptions(Args);
+ (Tool == ToolType::Strip) ? parseStripOptions(Args, reportWarning)
+ : ((Tool == ToolType::InstallNameTool)
----------------
Perhaps time to switch this to a `switch`?
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