Sema: Clean up ExprRewriter::visitParenExpr()

slavapestov · slavapestov · commit 06f6621cdcc2 · 2018-12-16T01:02:56.000-05:00
simplifyType() preserves ParenType sugar so we don't have to jump through
hoops here.
diff --git a/lib/Sema/CSApply.cpp b/lib/Sema/CSApply.cpp
@@ -2815,11 +2815,7 @@ namespace {
     }
 
     Expr *visitParenExpr(ParenExpr *expr) {
-      auto &ctx = cs.getASTContext();
-      auto pty = cs.getType(expr->getSubExpr());
-      cs.setType(expr, ParenType::get(ctx, pty->getInOutObjectType(),
-                                      ParameterTypeFlags().withInOut(pty->is<InOutType>())));
-      return expr;
+      return simplifyExprType(expr);
     }
 
     Expr *visitTupleExpr(TupleExpr *expr) {

Original file line number	Diff line number	Diff line change
`@@ -2815,11 +2815,7 @@ namespace {`
`2815`	`2815`	`}`
`2816`	`2816`
`2817`	`2817`	`Expr visitParenExpr(ParenExpr expr) {`
`2818`		`- auto &ctx = cs.getASTContext();`
`2819`		`- auto pty = cs.getType(expr->getSubExpr());`
`2820`		`- cs.setType(expr, ParenType::get(ctx, pty->getInOutObjectType(),`
`2821`		`- ParameterTypeFlags().withInOut(pty->is<InOutType>())));`
`2822`		`- return expr;`
	`2818`	`+ return simplifyExprType(expr);`
`2823`	`2819`	`}`
`2824`	`2820`
`2825`	`2821`	`Expr visitTupleExpr(TupleExpr expr) {`