Sema: Remove various calls to getAllConformances()

Author: slavapestov

lib/Sema/TypeCheckDecl.cpp

@@ -2700,8 +2700,6 @@ class DeclChecker : public DeclVisitor<DeclChecker> {
       checkEnumRawValues(TC, ED);
     }
 
-    ED->getAllConformances();
-
     TC.checkDeclCircularity(ED);
     TC.ConformanceContexts.push_back(ED);
   }
@@ -2725,8 +2723,6 @@ class DeclChecker : public DeclVisitor<DeclChecker> {
 
     checkAccessControl(TC, SD);
 
-    SD->getAllConformances();
-
     TC.checkDeclCircularity(SD);
     TC.ConformanceContexts.push_back(SD);
   }
@@ -2941,8 +2937,6 @@ class DeclChecker : public DeclVisitor<DeclChecker> {
       }
     }
 
-    CD->getAllConformances();
-
     TC.checkDeclAttributes(CD);
 
     checkInheritanceClause(CD);
@@ -3169,13 +3163,6 @@ class DeclChecker : public DeclVisitor<DeclChecker> {
       TC.checkDeclAttributes(ED);
 
     checkAccessControl(TC, ED);
-
-    // Trigger the creation of all of the conformances associated with this
-    // nominal type.
-    // FIXME: This is a hack to make sure that the type checker precomputes
-    // enough information for later passes that might query conformances.
-    if (auto nominal = ED->getExtendedNominal())
-      (void) nominal->getAllConformances();
   }
 
   void visitTopLevelCodeDecl(TopLevelCodeDecl *TLCD) {

lib/Sema/TypeChecker.cpp

@@ -334,10 +334,8 @@ static void typeCheckFunctionsAndExternalDecls(SourceFile &SF, TypeChecker &TC)
         TC.checkFunctionErrorHandling(AFD);
         continue;
       }
-      if (auto nominal = dyn_cast<NominalTypeDecl>(decl)) {
-        (void)nominal->getAllConformances();
+      if (isa<NominalTypeDecl>(decl))
         continue;
-      }
       if (isa<VarDecl>(decl))
         continue;
       llvm_unreachable("Unhandled external definition kind");