[PATCH] D43149: [analyzer] Fix a crash on destroying a temporary array.

2018-02-15 Thread Phabricator via Phabricator via cfe-commits
This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rC325286: [analyzer] Suppress temporary destructors for temporary arrays. (authored by dergachev, committed by ). Repositor

[PATCH] D43149: [analyzer] Fix a crash on destroying a temporary array.

2018-02-09 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ updated this revision to Diff 133718. NoQ added a comment. > And even then, calling a destructor of a single array element does not > invalidate the whole array for us, because destructors are `const` (unless > there are mutable members). So we'd have to do this manually later as well. Hmm,

[PATCH] D43149: [analyzer] Fix a crash on destroying a temporary array.

2018-02-09 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ created this revision. NoQ added reviewers: dcoughlin, xazax.hun, a.sidorin, george.karpenkov, szepet. Herald added subscribers: cfe-commits, rnkovacs. Tried to actually run the analyzer with temporary destructor support on some real code, found two crashes for now - https://reviews.llvm.org/