[PATCH] D36782: [Bash-autocompletion] Add support for static analyzer flags
Raphael Isemann via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Aug 22 07:25:07 PDT 2017
teemperor accepted this revision.
teemperor added a comment.
This revision is now accepted and ready to land.
Found one more minor comment typo. And could you do your changes to `OptParserEmitter.cpp` all in this patch? Because Rui/Me pointed out those things on this review, so this patch should also fix them IMHO :)
Otherwise this is good to go, nice work!
================
Comment at: llvm/include/llvm/Option/OptTable.h:152
+ /// "foo, bar..", where foo and bar is the argument which the Option flag
+ // takes
+ ///
----------------
`///` instead of `//`.
https://reviews.llvm.org/D36782
More information about the cfe-commits
mailing list