Merge pull request #59559 from slavapestov/sendable-mismatch-capture-lifetime-fix-5.7

Sema: Fix capture lifetime issue in protocol conformance sendable mismatch [5.7]

Author: slavapestov

lib/Sema/TypeCheckProtocol.cpp

@@ -4198,8 +4198,10 @@ ConformanceChecker::resolveWitnessViaLookup(ValueDecl *requirement) {
       if (behavior != DiagnosticBehavior::Ignore) {
         bool isError = behavior < DiagnosticBehavior::Warning;
 
+        // Avoid relying on the lifetime of 'this'.
+        const DeclContext *DC = this->DC;
         diagnoseOrDefer(requirement, isError,
-                        [this, requirement, witness, sendFrom](
+                        [DC, requirement, witness, sendFrom](
                           NormalProtocolConformance *conformance) {
           diagnoseSendabilityErrorBasedOn(conformance->getProtocol(), sendFrom,
                                           [&](DiagnosticBehavior limit) {

test/Concurrency/sendable_witness_check_delayed.swift

@@ -0,0 +1,16 @@
+// RUN: %target-typecheck-verify-swift
+
+// This triggers a conformance check with SuppressDiagnostics=true.
+let x = S().f {}
+
+protocol P {
+  associatedtype A
+
+  func f(_: A) -> Int // expected-note {{expected sendability to match requirement here}}
+}
+
+struct S : P {
+  typealias A = () -> ()
+  func f(_: @Sendable () -> ()) -> Int { return 0 }
+  // expected-warning@-1 {{sendability of function types in instance method 'f' does not match requirement in protocol 'P'}}
+}