[PATCH] D41795: [analyzer] Inline destructors for non-array deletes.

2018-01-09 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ abandoned this revision. NoQ added a comment. > because the new behavior of operator new is to return an ElementRegion > surrounding the void pointer The new behavior was reverted in https://reviews.llvm.org/D41250#971888 so this patch is no longer useful. Repository: rC Clang https://

[PATCH] D41795: [analyzer] Inline destructors for non-array deletes.

2018-01-05 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ added inline comments. Comment at: lib/StaticAnalyzer/Core/ExprEngineCallAndReturn.cpp:692-693 + if (const Stmt *DtorExpr = Dtor.getOriginExpr()) +if (const Stmt *ParentExpr = +CurLC->getParentMap().getParent(DtorExpr)) + if (const CXXDel

[PATCH] D41795: [analyzer] Inline destructors for non-array deletes.

2018-01-05 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, mehdi_amini. Similarly to how we allow (since https://reviews.llvm.org/D40560) inlining the constructor after `operator new` which isn't `ope