Merge pull request #29546 from LucianoPAlmeida/visit-try-expr

[CSDiag] Removing unused FailureDiagnosis::visitTryExpr

Author: xedin

lib/Sema/CSDiag.cpp

@@ -221,7 +221,6 @@ class FailureDiagnosis :public ASTVisitor<FailureDiagnosis, /*exprresult*/bool>{
   validateContextualType(Type contextualType, ContextualTypePurpose CTP);
 
   bool visitExpr(Expr *E);
-  bool visitTryExpr(TryExpr *E);
 
   bool visitApplyExpr(ApplyExpr *AE);
   bool visitRebindSelfInConstructorExpr(RebindSelfInConstructorExpr *E);
@@ -1425,12 +1424,6 @@ visitRebindSelfInConstructorExpr(RebindSelfInConstructorExpr *E) {
   return false;
 }
 
-/// A TryExpr doesn't change it's argument, nor does it change the contextual
-/// type.
-bool FailureDiagnosis::visitTryExpr(TryExpr *E) {
-  return visit(E->getSubExpr());
-}
-
 bool FailureDiagnosis::visitExpr(Expr *E) {
   // Check each of our immediate children to see if any of them are
   // independently invalid.