[PATCH] Fix representation of compound literals for C++ objects with destructors

Richard Smith richard at metafoo.co.uk
Fri May 3 19:04:36 PDT 2013


  LGTM


================
Comment at: lib/Sema/SemaInit.cpp:4508
@@ -4502,2 +4507,3 @@
     case InitializedEntity::EK_LambdaCapture:
+    case InitializedEntity::EK_CompoundLiteralInit:
       return false;
----------------
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.


http://llvm-reviews.chandlerc.com/D742



More information about the cfe-commits mailing list