[PATCH] Fix representation of compound literals for C++ objects with destructors
Jordan Rose
jordan_rose at apple.com
Fri May 3 19:07:51 PDT 2013
================
Comment at: lib/Sema/SemaInit.cpp:4508
@@ -4502,2 +4507,3 @@
case InitializedEntity::EK_LambdaCapture:
+ case InitializedEntity::EK_CompoundLiteralInit:
return false;
----------------
Richard Smith wrote:
> This looks wrong -- the temporary will be destroyed, even if we're not binding it as part of the initialization -- though I guess this doesn't matter since the relevant checks will be performed by the caller anyway.
Ah, it's an access/validity check for the destructor. Okay, I'll flip this one.
http://llvm-reviews.chandlerc.com/D742
More information about the cfe-commits
mailing list