[cfe-commits] [Patch] Update Sema Diagnostics to use %diff

Richard Trieu rtrieu at google.com
Thu Jul 12 14:22:08 PDT 2012


On Thu, Jul 12, 2012 at 2:16 PM, Chandler Carruth <chandlerc at google.com>wrote:

> Totally psyched about this going in...
>
> Meta comment:
>
> Can you fold all of the tests into a single file?
>
> Our test running infrastructure scales better with one larger file than
> with lots of smaller ones.
>

There's possibly some flag mismatch from going to a single file, but I can
definitely shrink the number of test files.

>
>
> On Thu, Jul 12, 2012 at 2:07 PM, Richard Trieu <rtrieu at google.com> wrote:
>
>> Change diagnostic messages in DiagnosticsSemaKinds.td to use %diff when
>> two QualType's are required.  This will enable template type diffing for
>> all Sema messages.  The default text of the diff matches the old message so
>> no test cases need to be updated.  Some tests have been added for the
>> modified messages.
>>
>> Patch attached and also available at:
>> http://llvm-reviews.chandlerc.com/D6
>>
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120712/97229dfa/attachment.html>


More information about the cfe-commits mailing list