fixup! fix incorrect completion ObjCObjectTypes

Regression introduced in `5f4c39b38976859d1fc219abe476a4075d11028e`

Author: Michael137

lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp

@@ -2646,13 +2646,9 @@ static clang::Type const *GetCompleteEnumType(clang::ASTContext *ast,
 }
 
 static clang::Type const *
-GetCompleteObjCInterfaceType(clang::ASTContext *ast, clang::QualType qual_type,
+GetCompleteObjCInterfaceType(clang::ASTContext *ast,
+                             clang::ObjCObjectType const *objc_class_type,
                              bool allow_completion = true) {
-  const clang::ObjCObjectType *objc_class_type =
-      llvm::dyn_cast<clang::ObjCObjectType>(qual_type);
-  if (!objc_class_type)
-    return nullptr;
-
   clang::ObjCInterfaceDecl *class_interface_decl =
       objc_class_type->getInterface();
   // We currently can't complete objective C types through the newly added
@@ -2712,8 +2708,13 @@ static bool GetCompleteQualType(clang::ASTContext *ast,
   } break;
   case clang::Type::ObjCObject:
   case clang::Type::ObjCInterface: {
-    if (auto const *ty = llvm::dyn_cast_or_null<ObjCInterfaceType>(
-            GetCompleteObjCInterfaceType(ast, qual_type, allow_completion)))
+    const clang::ObjCObjectType *objc_class_type =
+        llvm::dyn_cast<clang::ObjCObjectType>(qual_type);
+    if (!objc_class_type)
+      return true;
+
+    if (auto const *ty = GetCompleteObjCInterfaceType(ast, objc_class_type,
+                                                      allow_completion))
       return true;
 
     return false;