Skip to content

Add Builtin.hopToActor to the feature list. #38379

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
Jul 14, 2021

Conversation

atrick
Copy link
Contributor

@atrick atrick commented Jul 13, 2021

This builtin never occurs in @inlinable code. But apparently we still
need to add a language feature for every builtin. This must allow
older compilers to reparse the library source (though I don't know why
that would ever happen!)

Fixes rdar://80525569 error: module 'Builtin' has no member named 'hopToActor')

@atrick atrick requested a review from DougGregor July 13, 2021 23:25
@atrick atrick force-pushed the fix-builtin-hop branch 2 times, most recently from 93c5c36 to d0a2c0b Compare July 13, 2021 23:27
This builtin never occurs in @inlinable code. But apparently we still
need to add a language feature for every builtin. This must allow
older compilers to reparse the library source (though I don't know why
that would ever happen!)

Fixes rdar://80525569 error: module 'Builtin' has no member named 'hopToActor')
@atrick atrick force-pushed the fix-builtin-hop branch from d0a2c0b to 982b47d Compare July 13, 2021 23:28
@atrick
Copy link
Contributor Author

atrick commented Jul 13, 2021

@swift-ci test

@atrick
Copy link
Contributor Author

atrick commented Jul 14, 2021

@DougGregor Note that I'm checking for `>= 5.5' compiler here, and this Builtin was never implemented on 5.5. Presumably I need to cherry-pick this feature definition to 5.5 now even if I don't cherry-pick the builtin.

@atrick atrick merged commit 775840e into swiftlang:main Jul 14, 2021
@atrick atrick deleted the fix-builtin-hop branch July 14, 2021 06:28
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