[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