[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