[PATCH] Add a level parameter to ClangTidyCheck::diag.

Alp Toker alp at nuanti.com
Fri May 9 05:49:42 PDT 2014


On 08/05/2014 15:28, Alexander Kornienko wrote:
> On Thu, Apr 17, 2014 at 5:48 PM, Alexander Kornienko 
> <alexfh at google.com <mailto:alexfh at google.com>> wrote:
>
>     On Mon, Mar 3, 2014 at 4:26 PM, Alp Toker <alp at nuanti.com
>     <mailto:alp at nuanti.com>> wrote:
>
>
>         On 03/03/2014 15:02, Alp Toker wrote:
>
>
>             On 03/03/2014 12:53, Alexander Kornienko wrote:
>
>
>                 What do you think about this?
>
>
>             Keen to go ahead with it so long as we're treating the
>             work as much-needed cleanup, with clang-tools-extra
>             getting the functionality when it's ready. It is a
>             non-trivial but the reduction in cruft and will surely pay
>             off.
>
>
>         And to get a feel for things, I've attached is a testbed patch
>         for clang-tools-extra that generates diagnostics with tablegen
>         (concept only, not for review).
>
>         The generated diagnostics aren't usable yet but it does build.
>         Playing with this code gives an idea where we might want to go
>         and what remains to be done to make tablegenned diagnostics
>         usable cleanly by external modules.
>
>
>         Alp.
>
>
>     Hi Alp,
>
>     Do you expect to have free cycles to work on the infrastructure
>     part of the discussed changes any time soon?
>
>
> Ping.


Sure, I'll put aside some time to revisit this (currently looking at 
related issues in the diagnostic system).

Alp.


-- 
http://www.nuanti.com
the browser experts




More information about the cfe-commits mailing list