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

[Alexander Kornienko]

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


-- 
Best regards,
Alexander Kornienko
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120813/c2610250/attachment.html>