Merge pull request swiftlang#59672 from beccadax/what-is-the-protocol…

…-for-generating-sendable Ignore Sendable conformances in PrintAsClang
voidref · Jun 24, 2022 · dfb8d74 · dfb8d74
2 parents 158953c + 8a8e71e
commit dfb8d74
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/lib/PrintAsClang/ModuleContentsWriter.cpp b/lib/PrintAsClang/ModuleContentsWriter.cpp
@@ -439,8 +439,10 @@ class ModuleWriter {
     bool allRequirementsSatisfied = true;
 
     for (auto proto : PD->getInheritedProtocols()) {
-      assert(proto->isObjC());
-      allRequirementsSatisfied &= require(proto);
+      if (printer.shouldInclude(proto)) {
+        assert(proto->isObjC());
+        allRequirementsSatisfied &= require(proto);
+      }
     }
 
     if (!allRequirementsSatisfied)

diff --git a/test/PrintAsObjC/protocols.swift b/test/PrintAsObjC/protocols.swift
@@ -25,7 +25,7 @@ import objc_generics
 
 // CHECK-LABEL: @protocol B <A>
 // CHECK-NEXT: @end
-@objc protocol B : A {}
+@objc protocol B : A, Sendable {}
 
 // CHECK-LABEL: @protocol CompletionAndAsync
 // CHECK-NEXT: - (void)helloWithCompletion:(void (^ _Nonnull)(BOOL))completion;