[cfe-commits] r149127 - in /cfe/trunk: include/clang/AST/ include/clang/Parse/ include/clang/Sema/ lib/AST/ lib/Parse/ lib/Sema/ lib/Serialization/ tools/libclang/

[Enea Zaffanella]

On 01/27/2012 08:39 PM, Richard Smith wrote:
> Is there some way you can add tests for this?
>
> On Fri, Jan 27, 2012 at 1:46 AM, Abramo Bagnara
> <abramo.bagnara at gmail.com <mailto:abramo.bagnara at gmail.com>> wrote:
>
>     Author: abramo
>     Date: Fri Jan 27 03:46:47 2012
>     New Revision: 149127
>
>     URL: http://llvm.org/viewvc/llvm-project?rev=149127&view=rev
>     <http://llvm.org/viewvc/llvm-project?rev=149127&view=rev>
>     Log:
>     Added source location for the template keyword in AST template-id
>     expressions.


Short answer: no way that we are aware of.

Longer answer:
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

   http://lists.cs.uiuc.edu/pipermail/cfe-dev/2012-January/019564.html

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.

Enea.