You should be able to take the union of flags?<div><br></div><div>Alternatively, you could add these test cases to the files currently testing those warning flags, which might be cleaner.</div><div><br></div><div>I actually wonder if this test shouldn't just focus on a few "interesting" tests of complex type diffing, and you should add the basic test that a particular diagnostic employs type diffing to the existing files in the test suite that exercise that particular diagnostic.</div>


<div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jul 12, 2012 at 2:22 PM, Richard Trieu <span dir="ltr"><<a href="mailto:rtrieu@google.com" target="_blank">rtrieu@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="gmail_quote"><div class="im">On Thu, Jul 12, 2012 at 2:16 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

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

</blockquote><div><br></div></div><div>There's possibly some flag mismatch from going to a single file, but I can definitely shrink the number of test files.</div><div class="im"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div class="gmail_extra"><br><br><div class="gmail_quote"><div><div>On Thu, Jul 12, 2012 at 2:07 PM, Richard Trieu <span dir="ltr"><<a href="mailto:rtrieu@google.com" target="_blank">rtrieu@google.com</a>></span> wrote:<br>

</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>
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.<div>



<br></div><div>Patch attached and also available at:</div><div><a href="http://llvm-reviews.chandlerc.com/D6" target="_blank">http://llvm-reviews.chandlerc.com/D6</a></div>
<br></div></div>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>
</blockquote></div></div><br>
</blockquote></div><br></div>