[PATCH] D43144: [analyzer] Implement path notes for temporary destructors.

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 9 14:47:42 PST 2018


NoQ added inline comments.


================
Comment at: test/Analysis/inlining/temp-dtors-path-notes.cpp:17
+        // expected-note at -2{{Returning from constructor for 'C'}}
+        // expected-note at -3{{Calling '~C'}}
+}
----------------
george.karpenkov wrote:
> Should we have "returning from ~C"? At least we do have it for inlined functions.
Not here, because the bug occurs within `~C`. But we should definitely test it, yeah.


https://reviews.llvm.org/D43144





More information about the cfe-commits mailing list