Sema: validateDecl validates the context of GenericTypeParamDecls

Author: xymus

lib/Sema/TypeCheckDecl.cpp

@@ -3640,10 +3640,6 @@ static Type buildAddressorResultType(TypeChecker &TC,
 }
 
 void TypeChecker::validateDecl(ValueDecl *D) {
-  // Generic parameters are validated as part of their context.
-  if (isa<GenericTypeParamDecl>(D))
-    return;
-
   // Handling validation failure due to re-entrancy is left
   // up to the caller, who must call hasValidSignature() to
   // check that validateDecl() returned a fully-formed decl.
@@ -3681,6 +3677,10 @@ void TypeChecker::validateDecl(ValueDecl *D) {
       return;
   }
 
+  // Generic parameters are validated as part of their context.
+  if (isa<GenericTypeParamDecl>(D))
+    return;
+
   // Validating the parent may have triggered validation of this declaration,
   // so just return if that was the case.
   if (D->hasValidationStarted()) {