averello/swift-mirror
1
0
Fork 0
mirror of https://github.com/apple/swift.git synced 2025-12-21 12:14:44 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix symbol graph bug where enabling extension block format caused inheritsFrom relationships of extended symbols to be emitted in extension graphs (#61951)

Browse Source
This commit is contained in:
Max Obermeier
2022-11-15 17:18:09 +01:00
committed by GitHub
parent fc954d299b
commit ddf5cf944c
5 changed files with 38 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
lib/SymbolGraphGen/SymbolGraph.cpp
View File

@@ -377,7 +377,7 @@ void SymbolGraph::recordConformanceSynthesizedMemberRelationships(Symbol S) {
void void
SymbolGraph::recordInheritanceRelationships(Symbol S) { SymbolGraph::recordInheritanceRelationships(Symbol S) {
const auto VD = S.getSymbolDecl(); const auto VD = S.getLocalSymbolDecl();
if (const auto *NTD = dyn_cast<NominalTypeDecl>(VD)) { if (const auto *NTD = dyn_cast<NominalTypeDecl>(VD)) {
for (const auto &InheritanceLoc : NTD->getInherited()) { for (const auto &InheritanceLoc : NTD->getInherited()) {
auto Ty = InheritanceLoc.getType(); auto Ty = InheritanceLoc.getType();
@@ -390,7 +390,7 @@ SymbolGraph::recordInheritanceRelationships(Symbol S) {
continue; continue;
} }
recordEdge(Symbol(this, VD, nullptr), recordEdge(Symbol(this, NTD, nullptr),
Symbol(this, InheritedTypeDecl, nullptr), Symbol(this, InheritedTypeDecl, nullptr),
RelationshipKind::InheritsFrom()); RelationshipKind::InheritsFrom());
} }

11
test/SymbolGraph/Relationships/InheritsFrom.swift
View File

@@ -1,11 +0,0 @@
// RUN: %empty-directory(%t)
// RUN: %target-build-swift %s -module-name InheritsFrom -emit-module -emit-module-path %t/
// RUN: %target-swift-symbolgraph-extract -module-name InheritsFrom -I %t -pretty-print -output-dir %t
// RUN: %FileCheck %s --input-file %t/InheritsFrom.symbols.json
public class Base {}
public class Derived: Base {}
// CHECK: "kind": "inheritsFrom"
// CHECK-NEXT: "source": "s:12InheritsFrom7DerivedC"
// CHECK-NEXT: "target": "s:12InheritsFrom4BaseC"

11
test/SymbolGraph/Relationships/InheritsFrom/Basic.swift Normal file
View File

@@ -0,0 +1,11 @@
// RUN: %empty-directory(%t)
// RUN: %target-build-swift %s -module-name Basic -emit-module -emit-module-path %t/
// RUN: %target-swift-symbolgraph-extract -module-name Basic -I %t -pretty-print -output-dir %t
// RUN: %FileCheck %s --input-file %t/Basic.symbols.json
public class Base {}
public class Derived: Base {}
// CHECK: "kind": "inheritsFrom"
// CHECK-NEXT: "source": "s:5Basic7DerivedC"
// CHECK-NEXT: "target": "s:5Basic4BaseC"

23
test/SymbolGraph/Relationships/InheritsFrom/ExtensionBlockSymbol.swift Normal file
View File

@@ -0,0 +1,23 @@
// RUN: %empty-directory(%t)
// RUN: %target-build-swift %S/Inputs/Base.swift -module-name Base -emit-module -emit-module-path %t/
// RUN: %target-build-swift %s -module-name ExtensionBlockSymbol -emit-module -emit-module-path %t/ -I %t
// RUN: %target-swift-symbolgraph-extract -module-name ExtensionBlockSymbol -I %t -pretty-print -output-dir %t -emit-extension-block-symbols
// RUN: %target-swift-symbolgraph-extract -module-name Base -I %t -pretty-print -output-dir %t -emit-extension-block-symbols
// RUN: %FileCheck %s --input-file %t/Base.symbols.json --check-prefix BASE
// RUN: %FileCheck %s --input-file %t/ExtensionBlockSymbol.symbols.json --check-prefix LOCAL
// RUN: %FileCheck %s --input-file %t/ExtensionBlockSymbol@Base.symbols.json --check-prefix EXTENSION
import Base
public extension Derived {
func foo() {}
}
// BASE: "kind": "inheritsFrom"
// BASE-NEXT: "source": "s:4Base7DerivedC"
// BASE-NEXT: "target": "s:4BaseAAC"
// LOCAL-DAG: "symbols": []
// LOCAL-DAG: "relationships": []
// EXTENSION-NOT: "kind": "inheritsFrom"

2
test/SymbolGraph/Relationships/InheritsFrom/Inputs/Base.swift Normal file
View File

@@ -0,0 +1,2 @@
public class Base {}
public class Derived: Base {}