clang format + code review

Author: ganenkokb-yandex

clang/lib/AST/ASTImporter.cpp

@@ -4036,7 +4036,7 @@ ExpectedDecl ASTNodeImporter::VisitFunctionDecl(FunctionDecl *D) {
     // To avoid an infinite recursion when importing, create the FunctionDecl
     // with a simplified return type.
     if (hasReturnTypeDeclaredInside(D) ||
-      Importer.DeclTypeCycles.find(D) != Importer.DeclTypeCycles.end()) {
+        Importer.DeclTypeCycles.find(D) != Importer.DeclTypeCycles.end()) {
       FromReturnTy = Importer.getFromContext().VoidTy;
       UsedDifferentProtoType = true;
     }
@@ -4059,13 +4059,11 @@ ExpectedDecl ASTNodeImporter::VisitFunctionDecl(FunctionDecl *D) {
   }
 
   Error Err = Error::success();
-  if (!UsedDifferentProtoType) {
+  if (!UsedDifferentProtoType)
     Importer.DeclTypeCycles.insert(D);
-  }
   auto T = importChecked(Err, FromTy);
-  if (!UsedDifferentProtoType) {
+  if (!UsedDifferentProtoType)
     Importer.DeclTypeCycles.erase(D);
-  }
   auto TInfo = importChecked(Err, FromTSI);
   auto ToInnerLocStart = importChecked(Err, D->getInnerLocStart());
   auto ToEndLoc = importChecked(Err, D->getEndLoc());

clang/unittests/AST/ASTImporterTest.cpp

@@ -3249,7 +3249,8 @@ TEST_P(ImportExpr, CycleInAutoTemplateSpec) {
   }
 )";
   // This test reproduces the StrCatT recursion pattern with concepts and span
-  // that may cause infinite recursion during AST import due to circular dependencies
+  // that may cause infinite recursion during AST import due to circular
+  // dependencies
   testImport(Code, Lang_CXX20, "", Lang_CXX20, Verifier,
              functionTemplateDecl(hasName("declToImport")));
 }