[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