[Concurrency] Don't spuriously record protocols inherited by Actor.

Author: DougGregor

lib/Frontend/ModuleInterfaceSupport.cpp

@@ -467,15 +467,7 @@ class InheritedProtocolCollector {
         // not extensions of that 'actor class'.
         if (actorClass &&
             inherited->isSpecificProtocol(KnownProtocolKind::Actor))
-          return TypeWalker::Action::Continue; 
-
-#if false
-        // If the protocol is a marker protocol, print it separately.
-        if (inherited->isMarkerProtocol()) {
-          protocolsToPrint.push_back({inherited, protoAndAvailability.second});
           return TypeWalker::Action::SkipChildren;
-        }
-#endif
 
         if (inherited->isSPI() && !printOptions.PrintSPIs)
           return TypeWalker::Action::Continue;

test/ModuleInterface/actor_isolation.swift

@@ -1,8 +1,11 @@
 // RUN: %empty-directory(%t)
 // RUN: %target-swift-frontend -emit-module -o %t/Test.swiftmodule -emit-module-interface-path %t/Test.swiftinterface -module-name Test -enable-experimental-concurrency %s
 // RUN: %FileCheck %s --check-prefix FROMSOURCE --check-prefix CHECK < %t/Test.swiftinterface
+// RUN: %target-swift-frontend -typecheck-module-from-interface -module-name Test %t/Test.swiftinterface
+
 // RUN: %target-swift-frontend -emit-module -o /dev/null -merge-modules %t/Test.swiftmodule -disable-objc-attr-requires-foundation-module -emit-module-interface-path %t/TestFromModule.swiftinterface -module-name Test -enable-experimental-concurrency
 // RUN: %FileCheck %s --check-prefix FROMMODULE --check-prefix CHECK < %t/TestFromModule.swiftinterface
+// RUN: %target-swift-frontend -typecheck-module-from-interface -module-name Test %t/TestFromModule.swiftinterface
 
 // REQUIRES: concurrency
 

test/ModuleInterface/features.swift

@@ -77,9 +77,7 @@ public func runSomethingSomewhere(body: () async -> Void) { }
 // CHECK-NEXT: #endif
 public func stage(with actor: MyActor) { }
 
-// CHECK: #if compiler(>=5.3) && $MarkerProtocol
-// CHECK-NEXT: extension FeatureTest.MyActor : Swift.ConcurrentValue {}
-// CHECK-NEXT: #endif
+// CHECK-NOT: extension FeatureTest.MyActor : Swift.ConcurrentValue
 
 // CHECK: #if compiler(>=5.3) && $MarkerProtocol
 // CHECK-NEXT: extension FeatureTest.OldSchool : FeatureTest.MP {