Skip to content

[5.2][SourceKit] Avoid calling AbstractStorageDecl::isSettable during syntactic operations #30774

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

Conversation

akyrtzi
Copy link
Contributor

@akyrtzi akyrtzi commented Apr 2, 2020

isSettable can trigger typechecking and cause crashes during pure syntactic requests.

rdar://60441208

master: #30459

@akyrtzi akyrtzi requested a review from a team as a code owner April 2, 2020 19:59
@akyrtzi
Copy link
Contributor Author

akyrtzi commented Apr 2, 2020

@swift-ci Please test

@swift-ci
Copy link
Contributor

swift-ci commented Apr 2, 2020

Build failed
Swift Test Linux Platform
Git Sha - 3a2eaf66a47a93ca0996eb12bf92a379311f4660

@swift-ci
Copy link
Contributor

swift-ci commented Apr 2, 2020

Build failed
Swift Test OS X Platform
Git Sha - 3a2eaf66a47a93ca0996eb12bf92a379311f4660

@tkremenek
Copy link
Member

@swift-ci test

…ntactic operations

`isSettable` can trigger typechecking and cause crashes during pure syntactic requests.

rdar://60441208
@swift-ci
Copy link
Contributor

swift-ci commented Apr 2, 2020

Build failed
Swift Test Linux Platform
Git Sha - 3a2eaf66a47a93ca0996eb12bf92a379311f4660

@akyrtzi akyrtzi force-pushed the 5.2-avoid-isSettable-for-syntactic branch from 3a2eaf6 to e60c5c2 Compare April 2, 2020 23:47
@akyrtzi
Copy link
Contributor Author

akyrtzi commented Apr 2, 2020

@swift-ci Please test

@swiftlang swiftlang deleted a comment from swift-ci Apr 2, 2020
@swiftlang swiftlang deleted a comment from swift-ci Apr 2, 2020
@najacque najacque merged commit ff23a87 into swiftlang:swift-5.2-branch Apr 3, 2020
@akyrtzi akyrtzi deleted the 5.2-avoid-isSettable-for-syntactic branch April 3, 2020 03:27
@AnthonyLatsis AnthonyLatsis added swift 5.2 🍒 release cherry pick Flag: Release branch cherry picks labels Jan 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍒 release cherry pick Flag: Release branch cherry picks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants