diff options
| author | Tim Shen <timshen91@gmail.com> | 2016-06-09 21:13:39 +0000 |
|---|---|---|
| committer | Tim Shen <timshen91@gmail.com> | 2016-06-09 21:13:39 +0000 |
| commit | 17b3deeff3f0b246789c37911a445dbd4ad3456f (patch) | |
| tree | e03729ec0980541f07ba560ef329fb47b3443254 /clang/lib/AST/Expr.cpp | |
| parent | ee2ce4a50e16548cd4274e8532491cc8b06d7a8d (diff) | |
| download | bcm5719-llvm-17b3deeff3f0b246789c37911a445dbd4ad3456f.tar.gz bcm5719-llvm-17b3deeff3f0b246789c37911a445dbd4ad3456f.zip | |
Revert "[Temporary] Add an ExprWithCleanups for each C++ MaterializeTemporaryExpr."
This reverts r272296, since there are clang-tidy failures that appear to
be caused by this change.
llvm-svn: 272310
Diffstat (limited to 'clang/lib/AST/Expr.cpp')
| -rw-r--r-- | clang/lib/AST/Expr.cpp | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/clang/lib/AST/Expr.cpp b/clang/lib/AST/Expr.cpp index dc87c0a8e26..4c0492c3935 100644 --- a/clang/lib/AST/Expr.cpp +++ b/clang/lib/AST/Expr.cpp @@ -2890,6 +2890,7 @@ bool Expr::HasSideEffects(const ASTContext &Ctx, case CXXThrowExprClass: case CXXNewExprClass: case CXXDeleteExprClass: + case ExprWithCleanupsClass: case CoawaitExprClass: case CoyieldExprClass: // These always have a side-effect. @@ -2902,12 +2903,6 @@ bool Expr::HasSideEffects(const ASTContext &Ctx, return Finder.hasSideEffects(); } - case ExprWithCleanupsClass: - if (IncludePossibleEffects) - if (cast<ExprWithCleanups>(this)->cleanupsHaveSideEffects()) - return true; - break; - case ParenExprClass: case ArraySubscriptExprClass: case OMPArraySectionExprClass: |
