diff options
| author | Tim Shen <timshen91@gmail.com> | 2016-06-21 20:29:17 +0000 |
|---|---|---|
| committer | Tim Shen <timshen91@gmail.com> | 2016-06-21 20:29:17 +0000 |
| commit | 4a05bb8d8db64eedca81dea4493cae59d9c78d75 (patch) | |
| tree | 7ba2166506bd4aa950e5d0a82d17f791ca843ba6 /clang/lib/Analysis/Consumed.cpp | |
| parent | a4d156980e69fd3787a8a3948d4f1da16b1abf32 (diff) | |
| download | bcm5719-llvm-4a05bb8d8db64eedca81dea4493cae59d9c78d75.tar.gz bcm5719-llvm-4a05bb8d8db64eedca81dea4493cae59d9c78d75.zip | |
Re-commit "[Temporary] Add an ExprWithCleanups for each C++ MaterializeTemporaryExpr."
Since D21243 fixes relative clang-tidy tests.
This reverts commit a71d9fbd41e99def9159af2b01ef6509394eaeed.
llvm-svn: 273312
Diffstat (limited to 'clang/lib/Analysis/Consumed.cpp')
| -rw-r--r-- | clang/lib/Analysis/Consumed.cpp | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/clang/lib/Analysis/Consumed.cpp b/clang/lib/Analysis/Consumed.cpp index 9df23923b01..f2a7d904a54 100644 --- a/clang/lib/Analysis/Consumed.cpp +++ b/clang/lib/Analysis/Consumed.cpp @@ -466,9 +466,15 @@ class ConsumedStmtVisitor : public ConstStmtVisitor<ConsumedStmtVisitor> { MapType PropagationMap; InfoEntry findInfo(const Expr *E) { + if (auto Cleanups = dyn_cast<ExprWithCleanups>(E)) + if (!Cleanups->cleanupsHaveSideEffects()) + E = Cleanups->getSubExpr(); return PropagationMap.find(E->IgnoreParens()); } ConstInfoEntry findInfo(const Expr *E) const { + if (auto Cleanups = dyn_cast<ExprWithCleanups>(E)) + if (!Cleanups->cleanupsHaveSideEffects()) + E = Cleanups->getSubExpr(); return PropagationMap.find(E->IgnoreParens()); } void insertInfo(const Expr *E, const PropagationInfo &PI) { |
