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
Delete libSyntax #62145
Delete libSyntax #62145
Conversation
@swift-ci test |
@swift-ci test source compatibility |
@swift-ci please build toolchain windows platform |
Expand it to llvm::IntrusiveRefCntPtr
@swift-ci test |
@swift-ci test |
@swift-ci please build toolchain windows platform |
@swift-ci test source compatibility |
apple/llvm-project#5630 |
@swift-ci test |
Is this inadvertently conflicting with #62132? |
apple/swift-syntax#1084 |
@AnthonyLatsis @rintaro I suspect I’m going to win this race 😉. |
😅
|
Now that swift-syntax no longer consumes the serialized parse tree (see apple/swift-syntax#1041), and we are no longer building the internal swift-syntax dylib (see #61898), we are finally able to get rid of the C++ libSyntax library.
Delete libSyntax, the tests dependent upon it, and redistribute some of the remaining useful types it defined to other parts of the compiler.