[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 02/02/2012 11:48 PM, Eli Friedman wrote:
> On Fri, Jan 27, 2012 at 1:46 AM, Abramo Bagnara
> <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
>> Log:
>> Added source location for the template keyword in AST template-id expressions.

[...]

> It looks like this change had semantic side-effects; try the following
> testcase (derived from the gcc testsuite):
>
> struct A
> {
>    template<int I>
>    struct B {
>      static void b1();
>    };
> };
> template<int I>  void f2()
> {
>    A::template B<I>::template b1();
> }
> template void f2<0>();
>
> -Eli

Thanks for spotting this out.

I suspect this is not strictly due to a bug in commit 149127; rather, 
the commit has uncovered a latent issue in the code instantiating 
DependentScopeDeclRefExpr nodes.
We will try to provide a suitable fix asap.

Enea.