[PATCH] Proposal on how to fix temporary dtors.

Manuel Klimek klimek at google.com
Wed May 21 09:32:15 PDT 2014


Giving this a shot at pulling it together, and got a couple more questions...

================
Comment at: lib/StaticAnalyzer/Core/ExprEngine.cpp:54-56
@@ -53,1 +53,5 @@
 
+REGISTER_TRAIT_WITH_PROGRAMSTATE(
+    InitializedTemporariesSet,
+    llvm::ImmutableSet<const CXXBindTemporaryExpr *>);
+
----------------
Jordan Rose wrote:
> This needs to include the current StackFrameContext as well (for recursive functions). You can get that from the current LocationContext.
Can you elaborate on how I would put that into a datastructure? Just use a std::pair? (doesn't seem to work with ImmutableSet though)

Also, I seem unable to write a test that breaks because of this - any hints would be highly welcome.

http://reviews.llvm.org/D3627






More information about the cfe-commits mailing list