Apply suggestions from code review

Co-authored-by: Xiaodi Wu <[email protected]>

Author: calda

lib/Sema/MiscDiagnostics.cpp

@@ -1538,11 +1538,11 @@ static bool closureHasWeakSelfCapture(const AbstractClosureExpr *ACE) {
   return false;
 }
 
-/// Whether or not impliciy self decls in this closure require manual
+/// Whether or not implicit self decls in this closure require manual
 /// lookup in order to perform diagnostics with the semantics described
 /// in SE-0365. This is necessary for closures that capture self weakly
 /// in Swift 5 language modes, since implicit self decls in weak self
-//// closures has different semantics in Swift 5 than in Swift 6.
+/// closures have different semantics in Swift 5 than in Swift 6.
 static bool
 closureRequiresManualLookupForImplicitSelf(const AbstractClosureExpr *ACE) {
   if (ACE->getASTContext().LangOpts.isSwiftVersionAtLeast(6)) {
@@ -1689,7 +1689,7 @@ static void diagnoseImplicitSelfUseInClosure(const Expr *E,
       //   doVoidStuff { [weak self] in
       //     guard let self else { return }
       //     method()
-      //    }
+      //   }
       //
       // In Swift 6, the implicit self DRE refers to the self decl
       // defined in the guard statement. In Swift 5 mode, the implicit
@@ -2746,7 +2746,7 @@ class VarDeclUsageChecker : public ASTWalker {
       //   doVoidStuff { [weak self] in
       //     guard let self else { return }
       //     method()
-      //    }
+      //   }
       //
       // In Swift 6, the implicit self DRE refers to the self decl
       // defined in the guard statement. In Swift 5 mode, the implicit