Skip to content

[WIP][IDE]: surface inferred isolation information via sourcekit#89198

Draft
jamieQ wants to merge 1 commit into
swiftlang:mainfrom
jamieQ:sourcekit-isolation-info
Draft

[WIP][IDE]: surface inferred isolation information via sourcekit#89198
jamieQ wants to merge 1 commit into
swiftlang:mainfrom
jamieQ:sourcekit-isolation-info

Conversation

@jamieQ
Copy link
Copy Markdown
Contributor

@jamieQ jamieQ commented May 17, 2026

@jamieQ
Copy link
Copy Markdown
Contributor Author

jamieQ commented May 17, 2026

@swift-ci please smoke test

Out << "";
return;
case ActorIsolation::ActorInstance:
Out << "actor-isolated";
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure what the best thing to surface is here... In diagnostics this usually prints the variable name of the isolated param or whatever, which we could probably also do, but it would require some additional plumbing I think.

@jamieQ
Copy link
Copy Markdown
Contributor Author

jamieQ commented May 22, 2026

@ahoppen would you be the right person to take a look at this to let me know if this is roughly directionally correct? If not, who would you recommend I ask?

@ahoppen
Copy link
Copy Markdown
Member

ahoppen commented May 26, 2026

Thanks for opening the PR.

@rintaro and @hamishknight would be the points of contact here.

@jamieQ
Copy link
Copy Markdown
Contributor Author

jamieQ commented May 30, 2026

@rintaro @hamishknight is this something you'll be able to provide feedback on at some point? Please let me know if you'd prefer it be taken out of "draft" status for review if that matters for your process. Thanks in advance!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants