[cfe-commits] [PATCH] Parsing C++0x lambda expressions

John Freeman jfreeman at cse.tamu.edu
Thu Aug 4 06:54:08 PDT 2011


On 8/3/11 12:23 PM, David Blaikie wrote:
> I'd had the same thought when I was adding test cases&  fixing a
> couple of bugs, but when I went to rewrite it I wasn't confident
> enough that I could come up with a better solution. After reading this
> feedback I went back to have another go&  came up with the version
> I've attached (this includes everything I sent out in my last email,
> plus a couple more test cases around the effected paths).

I incorporated David's changes and tests into the attached patch. I 
moved CXX0XLambda* helper structs to Sema/DeclSpec.h. I feel they belong 
in Sema since they will be used in the lambda actions. I chose 
Sema/DeclSpec.h because they seemed Declarator-ish and I did not want to 
create a separate file unless it was necessary.

- John
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: lambda-parse.patch
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110804/04ae47c8/attachment.ksh>


More information about the cfe-commits mailing list