Remove a couple of unnecessary hasInterfaceType() checks

We call isInvalid() a few lines above, so the
interface type gets computed anyway.

Author: hamishknight

lib/Sema/CSSimplify.cpp

@@ -5274,10 +5274,6 @@ performMemberLookup(ConstraintKind constraintKind, DeclName memberName,
       return;
     }
 
-    // FIXME: Deal with broken recursion
-    if (!decl->hasInterfaceType())
-      return;
-
     // Dig out the instance type and figure out what members of the instance type
     // we are going to see.
     auto baseTy = candidate.getBaseType();
@@ -5649,10 +5645,6 @@ performMemberLookup(ConstraintKind constraintKind, DeclName memberName,
         return result;
       }
 
-      // FIXME: Deal with broken recursion
-      if (!cand->hasInterfaceType())
-        continue;
-
       result.addUnviable(getOverloadChoice(cand, /*isBridged=*/false,
                                            /*isUnwrappedOptional=*/false),
                          MemberLookupResult::UR_Inaccessible);