[PATCH] D87211: [llvm-objcopy] Consolidate and unify version tests

[Alexander Shaposhnikov via Phabricator via llvm-commits]

alexshap added inline comments.


================
Comment at: llvm/test/tools/llvm-objcopy/tool-version.test:8
+# RUN: llvm-install-name-tool --version | FileCheck %s
+# RUN: llvm-strip -V | FileCheck %s
+
----------------
jhenderson wrote:
> Presumably copy-paste error?
in fact, at the moment there is no alias "-V" for "--version" in InstallNameToolOpts.td,
i do not recall any  discussions around it, if i am not mistaken nobody has ever asked for it (cctools' install_name_tool doesn't have these flags that's why we have not run into any issues), but in general we can add this alias.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87211