[cfe-commits] [PATCH] Implemented -ast-dump, -ast-print, -ast-dump-filter options in clang-check

[Alexander Kornienko]

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 <alexfh at google.com>wrote:

> On Fri, Jul 27, 2012 at 3:40 PM, Manuel Klimek <klimek at google.com> 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 <alexfh at google.com>
>> 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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120809/b9b582a6/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: tooling-dump.diff
Type: application/octet-stream
Size: 3614 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120809/b9b582a6/attachment.obj>