[PATCH] D42991: [analyzer] Use EvalCallOptions for destructors as well.

2018-02-14 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 rL325209: [analyzer] Decide on inlining destructors via EvalCallOptions. (authored by dergachev, committed by ). Herald adde

[PATCH] D42991: [analyzer] Use EvalCallOptions for destructors as well.

2018-02-07 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ updated this revision to Diff 133299. NoQ added a comment. Remove the check in `shouldInlineCall()` as well. It's quite covered by the `IsConstructorWithImproperlyModeledTargetRegion` check. https://reviews.llvm.org/D42991 Files: include/clang/StaticAnalyzer/Core/PathSensitive/ExprEngine

[PATCH] D42991: [analyzer] Use EvalCallOptions for destructors as well.

2018-02-06 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ updated this revision to Diff 133106. NoQ added a comment. Remove a couple of accidental blank lines. https://reviews.llvm.org/D42991 Files: include/clang/StaticAnalyzer/Core/PathSensitive/ExprEngine.h lib/StaticAnalyzer/Core/ExprEngine.cpp lib/StaticAnalyzer/Core/ExprEngineCXX.cpp