[cfe-commits] r63868 - in /cfe/trunk/lib/AST: Decl.cpp ExprCXX.cpp

[Douglas Gregor]

On Feb 5, 2009, at 7:12 AM, Sebastian Redl wrote:

> Author: cornedbee
> Date: Thu Feb  5 09:12:41 2009
> New Revision: 63868
>
> URL: http://llvm.org/viewvc/llvm-project?rev=63868&view=rev
> Log:
> Fix the symptom of the regression, by having the  
> CXXConditionDeclExpr not destroy its Decl.
> However, the cause still remains: the Decl is linked into the chain  
> of its DeclContext and remains there despite being deleted.

I wouldn't worry about this much. CXXConditionDeclExpr doesn't seem  
like the right way to model the problem anyway. WhileStmts and IfStmts  
should probably just have an optional Decl* in their structure.

	- Doug