Revert change to ClassLoader::CreateTypeHandleForTypeDefThrowing

I originally thought the modification may assist debugging layout
issues as I did during the investigation. It however turns out
that basically the same class name check can be put in
EEClassLayoutInfo::CollectLayoutFieldMetadataThrowing so I am
removing this as superfluous temporary instrumentation.

Thanks

Tomas

Author: trylek

src/coreclr/vm/methodtablebuilder.cpp

@@ -12019,24 +12019,24 @@ ClassLoader::CreateTypeHandleForTypeDefThrowing(
         pAllocator,
         pamTracker);
 
-#ifdef _DEBUG
-    // Only MultiCastDelegate should inherit from Delegate
-    LPCUTF8 className;
-    LPCUTF8 nameSpace;
-    if (FAILED(pInternalImport->GetNameOfTypeDef(cl, &className, &nameSpace)))
-    {
-        className = nameSpace = "Invalid TypeDef record";
-    }
-#endif
-
     if ((pParentMethodTable != NULL) && (pParentMethodTable == g_pDelegateClass))
     {
         // Note we do not allow single cast delegates
         if (pModule->GetAssembly() != SystemDomain::SystemAssembly())
         {
             pAssembly->ThrowTypeLoadException(pInternalImport, cl, BFA_CANNOT_INHERIT_FROM_DELEGATE);
         }
+
+#ifdef _DEBUG
+        // Only MultiCastDelegate should inherit from Delegate
+        LPCUTF8 className;
+        LPCUTF8 nameSpace;
+        if (FAILED(pInternalImport->GetNameOfTypeDef(cl, &className, &nameSpace)))
+        {
+            className = nameSpace = "Invalid TypeDef record";
+        }
         BAD_FORMAT_NOTHROW_ASSERT(strcmp(className, "MulticastDelegate") == 0);
+#endif
     }
 
     if (fIsDelegate)