After non-static use of cl::opt was approved (r161751), this was unblocked. Committed as r161753.
On Thu, Aug 9, 2012 at 4:04 PM, Alexander Kornienko <[email protected]>wrote: > Update: implemented -ast-list option and added tests for all dump-related > options in clang-check. > > On Fri, Jul 27, 2012 at 3:48 PM, Alexander Kornienko <[email protected]>wrote: > >> On Fri, Jul 27, 2012 at 3:40 PM, Manuel Klimek <[email protected]> wrote: >> >>> Dependent on somebody confirming that: >>> a) using cl::opt as non-static variables is a supported use case >>> >> As for this, there's a unit-test waiting to be reviewed on the >> llvm-commits list: >> http://article.gmane.org/gmane.comp.compilers.llvm.cvs/117923/match=alexfh >> >> >>> b) we don't have a better option to add -extra-clang-flags that would >>> allow us to also specify -ast-dump* >>> ... this looks good to me. >>> >>> Chandler, opinions on either of those? >>> >>> On Fri, Jul 27, 2012 at 3:33 PM, Alexander Kornienko <[email protected]> >>> wrote: >>> > Hi all, >>> > >>> > This patch adds -ast-dump, -ast-print and -ast-dump-filter options to >>> > clang-check. It uses the existing implementation of these options and >>> only >>> > makes them available to use in clang-check. A test is included. >>> > >>> > Please, review this patch. Thanks! >>> > >>> > -- >>> > Best regards, >>> > Alexander Kornienko >>> >> >> -- >> Best regards, >> Alexander Kornienko >> > > > > -- > Best regards, > Alexander Kornienko > > -- Best regards, Alexander Kornienko
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
