rsmith added inline comments. ================ Comment at: lib/Sema/SemaInit.cpp:6197 @@ +6196,3 @@ + // but there may be a chance to merge them. + if (getLangOpts().CPlusPlus) + Cleanup.setExprNeedsCleanups(false); ---------------- Why is this C++-only? We presumably would want to clean up after materialized temporaries in other languages if they ever get created.
================ Comment at: lib/Sema/SemaInit.cpp:6461 @@ -6460,3 +6477,2 @@ - // If we're binding to an Objective-C object that has lifetime, we // need cleanups. Likewise if we're extending this temporary to automatic ---------------- Accidentally-deleted? http://reviews.llvm.org/D20498 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits