Skip to content

Commit 0a7b1cc

Browse files
committed
AST: Basically revert "Rename TypeTransform::transformSubMap()"
This undoes the effect of commit 865c040 since it introduced a performance regression.
1 parent b76047a commit 0a7b1cc

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

lib/AST/TypeSubstitution.cpp

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -329,7 +329,7 @@ class TypeSubstituter : public TypeTransform<TypeSubstituter> {
329329
std::optional<Type> transformLocalArchetypeType(LocalArchetypeType *local,
330330
TypePosition pos);
331331

332-
SubstitutionMap transformSubstitutionMap(SubstitutionMap subs);
332+
// SubstitutionMap transformSubstitutionMap(SubstitutionMap subs);
333333

334334
CanType transformSILField(CanType fieldTy, TypePosition pos);
335335
};
@@ -449,10 +449,13 @@ Type TypeSubstituter::transformDependentMemberType(DependentMemberType *dependen
449449
return result;
450450
}
451451

452+
// FIXME: This exposes a scalability issue; see test/SILGen/opaque_result_type_slow.swift.
453+
/*
452454
SubstitutionMap TypeSubstituter::transformSubstitutionMap(SubstitutionMap subs) {
453455
// FIXME: Take level into account? Move level down into IFS?
454456
return subs.subst(IFS);
455457
}
458+
*/
456459

457460
CanType TypeSubstituter::transformSILField(CanType fieldTy, TypePosition pos) {
458461
// Type substitution does not walk into the SILBoxType's field types, because

0 commit comments

Comments
 (0)