<div class="gmail_quote">On Fri, Jan 27, 2012 at 11:31 PM, Enea Zaffanella <span dir="ltr"><<a href="mailto:zaffanella@cs.unipr.it">zaffanella@cs.unipr.it</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On 01/27/2012 08:39 PM, Richard Smith wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
Is there some way you can add tests for this?<br>
<br>
On Fri, Jan 27, 2012 at 1:46 AM, Abramo Bagnara<br></div><div class="im">
<<a href="mailto:abramo.bagnara@gmail.com" target="_blank">abramo.bagnara@gmail.com</a> <mailto:<a href="mailto:abramo.bagnara@gmail.com" target="_blank">abramo.bagnara@gmail.<u></u>com</a>>> wrote:<br>
<br>
Author: abramo<br>
Date: Fri Jan 27 03:46:47 2012<br>
New Revision: 149127<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=149127&view=rev" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project?rev=149127&view=rev</a><br>
<<a href="http://llvm.org/viewvc/llvm-project?rev=149127&view=rev" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project?rev=149127&view=rev</a>><br>
Log:<br>
Added source location for the template keyword in AST template-id<br>
expressions.<br>
</div></blockquote>
<br>
<br>
Short answer: no way that we are aware of.<br>
<br>
Longer answer:<br>
the changes in both r149124 and r149127 are improving upon the "syntactic aspects" of the AST representation. Their effect can be observed by inspecting the output produced by -ast-print; see, for instance, the example code in<br>
<br>
<a href="http://lists.cs.uiuc.edu/pipermail/cfe-dev/2012-January/019564.html" target="_blank">http://lists.cs.uiuc.edu/<u></u>pipermail/cfe-dev/2012-<u></u>January/019564.html</a><br>
<br>
As far as we know, the output of -ast-print is just a convenience (probably, to simplify visual debugging of the AST). Nonetheless, the added info is important for some source-code based applications.<br><span class="HOEnZb"><font color="#888888"></font></span></blockquote>
</div><br><div>These tests would not be the first in the test-suite to run FileCheck over the output of -ast-print. That kind of test (while not ideal) seems much better than making large changes with no test coverage. That obviously won't check whether the source location is correct, but it will check whether it's valid in the right set of circumstances.</div>
<div><br></div><div>- Richard</div>