Sema: Don't finalize types defined in primary files

We already do everything in typeCheckDecl(), so there's no need
to walk the members of the type again.

Author: slavapestov

lib/Sema/TypeCheckDecl.cpp

@@ -4373,6 +4373,7 @@ class DeclChecker : public DeclVisitor<DeclChecker> {
 
     TC.validateDecl(ED);
     TC.DeclsToFinalize.remove(ED);
+    ED->setHasValidatedLayout();
 
     {
       // Check for circular inheritance of the raw type.
@@ -4406,6 +4407,7 @@ class DeclChecker : public DeclVisitor<DeclChecker> {
 
     TC.validateDecl(SD);
     TC.DeclsToFinalize.remove(SD);
+    SD->setHasValidatedLayout();
 
     TC.addImplicitConstructors(SD);
 
@@ -4531,6 +4533,7 @@ class DeclChecker : public DeclVisitor<DeclChecker> {
     TC.validateDecl(CD);
     TC.requestSuperclassLayout(CD);
     TC.DeclsToFinalize.remove(CD);
+    CD->setHasValidatedLayout();
 
     {
       // Check for circular inheritance.
@@ -6911,7 +6914,7 @@ void TypeChecker::validateDecl(ValueDecl *D) {
     }
 
     if (!isa<ClassDecl>(nominal))
-      DeclsToFinalize.insert(nominal);
+      requestNominalLayout(nominal);
 
     break;
   }