Add a Swift parser library that is written in Swift#616
Merged
DougGregor merged 3 commits intomainfrom Aug 22, 2022
Merged
Conversation
This starts an effort to implement the parser for the Swift compiler in Swift itself. Co-authored-by: Robert Widmann <devteam.codafi@gmail.com> Co-authored-by: Doug Gregor <dgregor@apple.com> Co-authored-by: Rintaro Ishizaki <rishizaki@apple.com>
Member
Author
|
@swift-ci Please test |
CodaFi
approved these changes
Aug 22, 2022
rintaro
approved these changes
Aug 22, 2022
Member
|
@compnerd, if you have a cycle or two, it's probably worth making sure this works on Windows. |
DougGregor
approved these changes
Aug 22, 2022
zoecarver
approved these changes
Aug 22, 2022
Member
|
Doesn't seem to build on Windows (which is a regression). At the very least we will need swiftlang/swift-docc-plugin#26. |
Contributor
|
The docc plugin is not required for this repository in general, and is not a dependency for this PR specifically. Docc support is built into Xcode, I am not sure why we have a plugin dependency as well. |
Member
|
I couldn't answer why we have that. We should invert the condition so that it is not used by default. |
Member
|
I'm fine with removing the DocC plugin dependency or inverting the condition. We shouldn't land this in a form that breaks the Windows build. |
Contributor
|
Let's strike it for now. We shouldn't have merged it without verifying that it worked for Windows |
compnerd
reviewed
Aug 22, 2022
Member
There was a problem hiding this comment.
This at least allows this PR to be built on Windows with:
swift build -Xcc -IC:\Library\Developer\Toolchains\unknown-Asserts-development.xctoolchain\usr\include
Note that building this requires a locally built toolchain as there was a fix for SPM that is needed that has not had a new toolchain snapshot built due to other failures introduced by the rebranch.
Thank you @compnerd! Co-authored-by: Saleem Abdulrasool <compnerd@compnerd.org>
Member
|
@swift-ci please test |
1 similar comment
Member
|
@swift-ci please test |
Member
|
@swift-ci please test |
benlangmuir
reviewed
Aug 22, 2022
| @@ -0,0 +1,70 @@ | |||
| ## Overview | |||
Contributor
There was a problem hiding this comment.
(not a blocker) The existence of this file triggers a warning, at least with 5.7:
warning: 'swift-syntax': found 1 file(s) which are unhandled; explicitly declare them as resources or exclude from the target
Sources/SwiftParser/README.md
Member
There was a problem hiding this comment.
I'll merge and we'll fix it up afterward. Thanks!
|
Nice. Swift would really benefit from code generating it's own code rather than C++. |
|
I can assure you that this does currently not work on Windows. I just tried using this framework in an SPM project on Windows. I am getting the following error: |
Member
Please use a build from main. At the very least, a recent snapshot. |
|
I used the most recent version. As far as I am concerned, this is the most recent snapshot |
Member
|
5.6 should be buildable - https://github.com/compnerd/swift-build/releases/tag/swift-format-5.6-RELEASE was built and would pull in swift-syntax 5.6. I wonder if the .1 has anything to do with it 🤔 |
Member
|
@broken-bytes If you're looking to try the new parser that's the subject of this pull request, you'll need |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
10 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This starts an effort to implement the parser for the Swift compiler in Swift itself.
For more information, please see the announcement forum post at: https://forums.swift.org/t/a-new-swift-parser-for-swiftsyntax/59813