[PATCH] D59746: [CommandLineParser] Add DefaultOption flag

Thu Apr 11 07:12:30 PDT 2019

hintonda added a comment.

In D59746#1462352 <https://reviews.llvm.org/D59746#1462352>, @jhenderson wrote:

> I've not looked at the implementation in depth, but cl::DefaultOption seems like a nice way of handling this. Please make sure that there is testing in llvm-readobj and llvm-objdump that test their own short alias interpretation somewhere though.

While I'm still working on unittests, I have added a few tests that should cover this.  Please let me know if you have any concerns.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D59746