[PATCH] D101483: [llvm-objdump] a -v alias for --version

[Jordan Rupprecht via Phabricator via llvm-commits]

rupprecht accepted this revision.
rupprecht added inline comments.


================
Comment at: llvm/test/tools/llvm-objdump/version.test:4
+
+CHECK: LLVM
+CHECK: Registered Targets:
----------------
MaskRay wrote:
> `CHECK: LLVM version`
> 
> 
> (While llvm-readobj/basic.test says the "LLVM" can be changed by cmake PACKAGE_NAME, I used "LLVM version" for llvm-symbolizer and it seems fine so far. )
"LLVM version" is fine. Just don't go further than that -- e.g. don't check "LLVM version XX.Y"


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101483