[cfe-commits] r147837 - in /cfe/trunk: include/clang/AST/DeclCXX.h lib/Sema/Sema.cpp

[Eli Friedman]

On Mon, Jan 9, 2012 at 8:24 PM, John McCall <rjmccall at apple.com> wrote:
> On Jan 9, 2012, at 7:38 PM, Eli Friedman wrote:
>> Author: efriedma
>> Date: Mon Jan  9 21:38:46 2012
>> New Revision: 147837
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=147837&view=rev
>> Log:
>> Per John's comment, it makes sense to ask isLambda on any CXXRecordDecl; make sure that's safe.  Get rid of a check which is now unnecessary in Sema::getFunctionLevelDeclContext().
>
> You could also just start the definition as soon as you start processing the lambda expression.

I do; the extra check is necessary to handle non-lambda CXXRecordDecls
without a definition.

-Eli