Skip to content

ASTScope: Stop sorting declarations by source range #34226

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

Conversation

slavapestov
Copy link
Contributor

ASTScope would filter out duplicate AST nodes and in some places, sort AST nodes by source range. Let's remove these and fix the underlying issues instead.

Builds upon #34207.

@slavapestov
Copy link
Contributor Author

swiftlang/llvm-project#1916
@swift-ci Please smoke test

@slavapestov
Copy link
Contributor Author

@swift-ci Please test source compatibility

@slavapestov slavapestov changed the title ASTScope: Stop sorting declarations by sort range ASTScope: Stop sorting declarations by source range Oct 8, 2020
@slavapestov slavapestov force-pushed the astscope-source-range-sorting branch from fe223bb to 14620a3 Compare October 8, 2020 03:28
@slavapestov slavapestov requested a review from davidungar October 8, 2020 03:29
@slavapestov
Copy link
Contributor Author

swiftlang/llvm-project#1916
@swift-ci Please smoke test

Copy link
Contributor

@davidungar davidungar left a comment

Choose a reason for hiding this comment

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

Looks fine! I think we discussed adding a general assertion to the AST to ensure that source ranges were in order everwhere.

@slavapestov
Copy link
Contributor Author

@swift-ci Please test Windows

@slavapestov slavapestov merged commit cb70220 into swiftlang:main Oct 8, 2020
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