Skip to content

[Distributed] fix missing _remote func entry in TBD #39499

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 1, 2021

Conversation

ktoso
Copy link
Contributor

@ktoso ktoso commented Sep 29, 2021

Resolves rdar://82199717

This reliably reproduces the issue of:

<unknown>:0: error: symbol '$s4test20SomeDistributedActorC5hello4nameS2S_tFTd' (super test.SomeDistributedActor.hello(name: Swift.String) -> Swift.String) is in generated IR file, but not in TBD file
<unknown>:0: error: symbol '$s4test20SomeDistributedActorC5hello4nameS2S_tFTdTu' (async function pointer to super test.SomeDistributedActor.hello(name: Swift.String) -> Swift.String) is in generated IR file, but not in TBD file

I can't really figure out how to emit those while I get a $s4test20SomeDistributedActorC5hello4nameS2S_tF in the TBDGenVisitor, does it mean the contents it is visiting are missing the Td method descriptor...?

@ktoso ktoso added the distributed Feature → concurrency: distributed actor label Sep 29, 2021
@ktoso ktoso force-pushed the wip-tbd-fix-distributed branch 3 times, most recently from 2e6ab1f to 4fd022a Compare October 1, 2021 02:06
@ktoso ktoso force-pushed the wip-tbd-fix-distributed branch from 4fd022a to 6834f61 Compare October 1, 2021 02:11
@ktoso
Copy link
Contributor Author

ktoso commented Oct 1, 2021

@swift-ci please smoke test and merge

@ktoso
Copy link
Contributor Author

ktoso commented Oct 1, 2021

@swift-ci please smoke test Linux

@ktoso ktoso merged commit 783b619 into swiftlang:main Oct 1, 2021
@ktoso ktoso deleted the wip-tbd-fix-distributed branch October 1, 2021 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
distributed Feature → concurrency: distributed actor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant