Skip to content

Commit 17572ce

Browse files
committed
[SILGen] Eliminate useConformancesFromSubstitutions() on SubstitutionList.
Update one caller first.
1 parent 921d41c commit 17572ce

File tree

3 files changed

+1
-13
lines changed

3 files changed

+1
-13
lines changed

lib/SILGen/SILGen.cpp

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1383,14 +1383,6 @@ void SILGenModule::useConformance(ProtocolConformanceRef conformanceRef) {
13831383
usedConformances.insert(root);
13841384
}
13851385

1386-
void
1387-
SILGenModule::useConformancesFromSubstitutions(SubstitutionList subs) {
1388-
for (auto &sub : subs) {
1389-
for (auto conformance : sub.getConformances())
1390-
useConformance(conformance);
1391-
}
1392-
}
1393-
13941386
void SILGenModule::useConformancesFromSubstitutions(
13951387
const SubstitutionMap subs) {
13961388
for (auto conf : subs.getConformances())

lib/SILGen/SILGen.h

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -425,9 +425,6 @@ class LLVM_LIBRARY_VISIBILITY SILGenModule : public ASTVisitor<SILGenModule> {
425425
/// it's in our TU.
426426
void useConformance(ProtocolConformanceRef conformance);
427427

428-
/// Mark protocol conformances from the given set of substitutions as used.
429-
void useConformancesFromSubstitutions(SubstitutionList subs);
430-
431428
/// Mark protocol conformances from the given set of substitutions as used.
432429
void useConformancesFromSubstitutions(SubstitutionMap subs);
433430

lib/SILGen/SILGenBuilder.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,8 +77,7 @@ MetatypeInst *SILGenBuilder::createMetatype(SILLocation loc, SILType metatype) {
7777
auto subMap = t->getContextSubstitutionMap(getSILGenModule().SwiftModule,
7878
decl);
7979
SmallVector<Substitution, 4> subs;
80-
genericSig->getSubstitutions(subMap, subs);
81-
getSILGenModule().useConformancesFromSubstitutions(subs);
80+
getSILGenModule().useConformancesFromSubstitutions(subMap);
8281
return false;
8382
});
8483

0 commit comments

Comments
 (0)