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

John McCall rjmccall at apple.com
Mon Jan 9 20:24:21 PST 2012


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.  That's probably good for the invariants anyway, since otherwise you'd be processing a method declared on a class without a definition, which is not otherwise possible.

John.



More information about the cfe-commits mailing list