[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