Skip to content

Commit 3495937

Browse files
authored
Merge pull request #40474 from apple/revert-40440-prefer-swift-type
Revert "lookup: prefer Swift types over Clang ones when multiple types are present in lookup"
2 parents a3cc692 + 0468b07 commit 3495937

File tree

1 file changed

+0
-18
lines changed

1 file changed

+0
-18
lines changed

lib/Sema/TypeCheckType.cpp

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1423,21 +1423,6 @@ static void diagnoseAmbiguousMemberType(Type baseTy, SourceRange baseRange,
14231423
}
14241424
}
14251425

1426-
static LookupTypeResult preferSwiftTypes(LookupTypeResult results) {
1427-
if (results.size() == 1)
1428-
return results;
1429-
LookupTypeResult newResults;
1430-
for (auto cur: results) {
1431-
if (cur.Member->getClangNode()) {
1432-
continue;
1433-
}
1434-
newResults.addResult(cur);
1435-
}
1436-
if (newResults.size() == 1)
1437-
return newResults;
1438-
return results;
1439-
}
1440-
14411426
/// Resolve the given identifier type representation as a qualified
14421427
/// lookup within the given parent type, returning the type it
14431428
/// references.
@@ -1548,9 +1533,6 @@ static Type resolveNestedIdentTypeComponent(TypeResolution resolution,
15481533
memberTypes = TypeChecker::lookupMemberType(
15491534
DC, parentTy, comp->getNameRef(), lookupOptions);
15501535

1551-
// Prefer Swift types over Clang types.
1552-
memberTypes = preferSwiftTypes(memberTypes);
1553-
15541536
// Name lookup was ambiguous. Complain.
15551537
// FIXME: Could try to apply generic arguments first, and see whether
15561538
// that resolves things. But do we really want that to succeed?

0 commit comments

Comments
 (0)