Skip to content

Commit 81bf449

Browse files
committed
AST: Re-enable TypeSubstituter::transformSubstitutionMap() again
Now that we cache substituted substitution maps inside the InFlightSubstitution, we can re-enable this.
1 parent 8d05d79 commit 81bf449

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

lib/AST/TypeSubstitution.cpp

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -349,7 +349,7 @@ class TypeSubstituter : public TypeTransform<TypeSubstituter> {
349349
std::optional<Type> transformLocalArchetypeType(LocalArchetypeType *local,
350350
TypePosition pos);
351351

352-
// SubstitutionMap transformSubstitutionMap(SubstitutionMap subs);
352+
SubstitutionMap transformSubstitutionMap(SubstitutionMap subs);
353353

354354
CanType transformSILField(CanType fieldTy, TypePosition pos);
355355
};
@@ -465,13 +465,10 @@ Type TypeSubstituter::transformDependentMemberType(DependentMemberType *dependen
465465
return result;
466466
}
467467

468-
// FIXME: This exposes a scalability issue; see test/SILGen/opaque_result_type_slow.swift.
469-
/*
470468
SubstitutionMap TypeSubstituter::transformSubstitutionMap(SubstitutionMap subs) {
471469
// FIXME: Take level into account? Move level down into IFS?
472470
return subs.subst(IFS);
473471
}
474-
*/
475472

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

0 commit comments

Comments
 (0)