PR #34476 [cxx-interop] Fix SR-13785

[cxx-interop] Fix SR-13785 #34476

CodaFi merged 13 commits into swiftlang:main from scentini:implementation-only-symbol-visibility-fix

scentini changed the title ~~[cxx-interop]~~ [cxx-interop] Fix SR-13785 5 years ago

Add a test that verifies that Swift correctly recognizes symbol visib…

954d3e6e

Fix header guards

00d4d118

Fix SR-13785

53c4393f

Minor fixes

e20b0f87

Move logic to TypeCheckAccess.getDisallowedOriginKind

a22e65e3

Add more tests

679ce361

scentini force pushed from e1ec45e3 to 679ce361 5 years ago

Format files

8062bfcb

Format files

81c07216

Change logic for visible decls lookup

3ca701e5

Merge remote-tracking branch 'upstream/main' into implementation-only…

da2a0662

Remove unnecessary header

58ff4023

Polish PR

1a18225e

scentini marked this pull request as ready for review 5 years ago

scentini requested a review from

CodaFi 5 years ago

scentini added c++ interop

Make use of ModuleDecl::isImportedImplementationOnly() when searching…

6898aae7

CodaFi approved these changes on 2020-11-23

CodaFi merged 356946f0 into main 5 years ago

scentini deleted the implementation-only-symbol-visibility-fix branch 5 years ago

Reviewers

CodaFi

Assignees

No one assigned

Labels

c++ interop

Milestone

No milestone