[cfe-commits] r129700 - in /cfe/trunk: lib/Analysis/CFG.cpp lib/Sema/SemaStmt.cpp test/Analysis/PR9741.cpp

[Douglas Gregor]

On Apr 18, 2011, at 9:39 AM, Richard Smith wrote:

> On Mon, April 18, 2011 17:16, Douglas Gregor wrote:
>> On Apr 18, 2011, at 8:49 AM, Richard Smith wrote:
>>> Author: rsmith
>>> Date: Mon Apr 18 10:49:25 2011
>>> New Revision: 129700
>>> 
>>> 
>>> URL: http://llvm.org/viewvc/llvm-project?rev=129700&view=rev
>>> Log:
>>> Fix PR9741. The implicit declarations created for range-based for loops
>>> weren't being added to the DeclContext (nor were they being marked as
>>> implicit). Also, the declarations were being emitted in the wrong order
>>> when building the CFG.
>> 
>> Are the declarations properly getting added to the DeclContext during
>> template instantiation, too?
> 
> Yes. The template case did not suffer from this bug: instantiating the
> VarDecl already adds the declaration to the DeclContext.

Okay!

	- Doug