Merge pull request #78986 from kubamracek/embedded-api-descriptor

[embedded] Don't emit extensions to API descriptor under Embedded Swift, avoid compiler crash

Author: kubamracek

lib/SIL/IR/SILSymbolVisitor.cpp

@@ -741,6 +741,9 @@ class SILSymbolVisitorImpl : public ASTVisitor<SILSymbolVisitorImpl> {
     if (canSkipNominal(nominal))
       return;
 
+    if (ED->getASTContext().LangOpts.hasFeature(Feature::Embedded))
+      return;
+
     if (auto CD = dyn_cast_or_null<ClassDecl>(ED->getImplementedObjCDecl())) {
       // @_objcImplementation extensions generate the class metadata symbols.
       (void)addClassMetadata(CD);

test/embedded/api-descriptor-crash.swift

@@ -0,0 +1,26 @@
+// RUN: %target-swift-frontend -emit-ir %s -enable-experimental-feature Embedded -emit-api-descriptor-path %t.json
+// RUN: %validate-json %t.json | %FileCheck %s
+
+// REQUIRES: swift_in_compiler
+// REQUIRES: optimized_stdlib
+// REQUIRES: swift_feature_Embedded
+
+public struct MyStruct {
+}
+
+extension MyStruct: Equatable {
+   public static func == (lhs: Self, rhs: Self) -> Bool { return false }
+}
+
+// CHECK:      "target"
+// CHECK-NEXT: "globals": [
+// CHECK-NEXT:   {
+// CHECK-NEXT:     "name": "{{(_main)|(main)}}",
+// CHECK-NEXT:     "access": "public",
+// CHECK-NEXT:     "file": "",
+// CHECK-NEXT:     "linkage": "exported"
+// CHECK-NEXT:   }
+// CHECK-NEXT: ],
+// CHECK-NEXT: "interfaces": [],
+// CHECK-NEXT: "categories": [],
+// CHECK-NEXT: "version": "1.0"