Merge pull request swiftlang#23744 from aschwaighofer/silgen_keypath_not_external_for_non_public

SILGen: Only set the external decl of a key path component if the accessor is public

Author: aschwaighofer

lib/SILGen/SILGenExpr.cpp

@@ -3217,8 +3217,7 @@ SILGenModule::emitKeyPathComponentForDecl(SILLocation loc,
                                 bool forPropertyDescriptor) {
   /// Returns true if a key path component for the given property or
   /// subscript should be externally referenced.
-  auto shouldUseExternalKeyPathComponent =
-    [&]() -> bool {
+  auto shouldUseExternalKeyPathComponent = [&]() -> bool {
     return (!forPropertyDescriptor &&
             (storage->getModuleContext() != SwiftModule ||
              storage->isResilient(SwiftModule, expansion)) &&
@@ -3227,11 +3226,15 @@ SILGenModule::emitKeyPathComponentForDecl(SILLocation loc,
             // Properties that only dispatch via ObjC lookup do not have nor
             // need property descriptors, since the selector identifies the
             // storage.
+            // Properties that are not public don't need property descriptors
+            // either.
             (!storage->hasAnyAccessors() ||
-             !getAccessorDeclRef(getRepresentativeAccessorForKeyPath(storage))
-             .isForeign));
-    };
-  
+             (!getAccessorDeclRef(getRepresentativeAccessorForKeyPath(storage))
+                   .isForeign &&
+              getAccessorDeclRef(getRepresentativeAccessorForKeyPath(storage))
+                      .getLinkage(ForDefinition) <= SILLinkage::PublicNonABI)));
+  };
+
   auto strategy = storage->getAccessStrategy(AccessSemantics::Ordinary,
                                              storage->supportsMutation()
                                                ? AccessKind::ReadWrite

test/IRGen/Inputs/keypaths_c_types.h

@@ -0,0 +1,5 @@
+union c_union {
+  struct some_struct {
+    void* data;
+  } some_field;
+};

test/IRGen/keypaths_c_types.swift

@@ -0,0 +1,8 @@
+// RUN: %target-swift-frontend -primary-file %s -emit-ir -import-objc-header %S/Inputs/keypaths_c_types.h
+
+// This used to crash while trying to emit a reference to the property
+// descriptor for the some_field property.
+
+struct Foo {
+  static let somePath: WritableKeyPath<c_union, some_struct>? = \c_union.some_field
+}

test/SILGen/Inputs/keypaths_objc.h

@@ -5,3 +5,9 @@
 @property(readonly) NSString *_Nonnull objcProp;
 
 @end
+
+union c_union {
+  struct some_struct {
+    void* data;
+  } some_field;
+};

test/SILGen/keypaths_objc.swift

@@ -96,3 +96,9 @@ func externalObjCProperty() {
   // CHECK-NOT: external #NSObject.description
   _ = \NSObject.description
 }
+
+func sharedCProperty() {
+  // CHECK:  keypath $WritableKeyPath<c_union, some_struct>
+  // CHECK-NOT: external #c_union.some_field
+  let dataKeyPath: WritableKeyPath<c_union, some_struct>? = \c_union.some_field
+}