[PATCH] D26839: [analyzer] An attempt to fix pr19539 - crashes on temporaries life-extended via members

Artem Dergachev via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 18 00:22:42 PST 2016


NoQ marked an inline comment as done.
NoQ added inline comments.


================
Comment at: test/Analysis/lifetime-extension.cpp:11
+  int j[2];
+  S s;
+  A() : i(1) {
----------------
alexshap wrote:
> what is the role of S in this test ?
I copy-pasted this from the original bug report. The only purpose of S is to provide a non-trivial destructor. I also thought it'd be a nice twist to the test to make the destructor non-obvious, but it doesn't really matter. Removed.


https://reviews.llvm.org/D26839





More information about the cfe-commits mailing list