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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
