Skip to content

Commit 82d4d1e

Browse files
authored
Merge pull request swiftlang#18817 from DougGregor/finalize-witnesses
2 parents 9856e51 + bb6272b commit 82d4d1e

File tree

3 files changed

+23
-0
lines changed

3 files changed

+23
-0
lines changed

lib/Sema/TypeCheckProtocol.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3486,6 +3486,10 @@ void ConformanceChecker::checkConformance(MissingWitnessDiagnosisKind Kind) {
34863486
auto witness = Conformance->getWitness(requirement, nullptr).getDecl();
34873487
if (!witness) return;
34883488

3489+
// Make sure that we finalize the witness, so we can emit this
3490+
// witness table.
3491+
TC.DeclsToFinalize.insert(witness);
3492+
34893493
// Objective-C checking for @objc requirements.
34903494
if (requirement->isObjC() &&
34913495
requirement->getFullName() == witness->getFullName() &&
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
import Foundation
2+
3+
extension C {
4+
@objc func foo(_: String) { }
5+
}
Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
// RUN: %target-swift-frontend -module-name test -emit-ir -primary-file %s %S/Inputs/require-finalize-witness-other.swift -sdk %sdk -o - | %FileCheck %s
2+
3+
// REQUIRES: objc_interop
4+
5+
import Foundation
6+
7+
@objc class C: NSObject { }
8+
9+
protocol P {
10+
func foo(_: String)
11+
}
12+
13+
// CHECK: define {{.*}} @"$S4test1CCAA1PA2aDP3fooyySSFTW"
14+
extension C: P { }

0 commit comments

Comments
 (0)