This is an archive of the discontinued LLVM Phabricator instance.

[llvm-install-name-tool] Add -V flag
ClosedPublic

Authored by alexander-shaposhnikov on Sep 7 2020, 9:15 PM.

Details

Summary

This diff adds -V alias for --version to make llvm-install-name-tool consistent with other tools (llvm-objcopy, llvm-strip, etc).

Test plan: make check-all

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald Transcript
alexander-shaposhnikov requested review of this revision.Sep 7 2020, 9:15 PM
MaskRay accepted this revision.Sep 7 2020, 9:37 PM

LGTM.

llvm/tools/llvm-objcopy/InstallNameToolOpts.td
36

(Alias may not need a line wrap)

This revision is now accepted and ready to land.Sep 7 2020, 9:37 PM
jhenderson accepted this revision.Sep 8 2020, 12:31 AM

LGTM too. I noticed that --version (and -V) are missing from the Command Guide for llvm-install-name-tool, along with --help, unlike e.g. llvm-objcopy. Perhaps worth a separate patch if you get a few minutes to fix the doc.

This revision was automatically updated to reflect the committed changes.