New Go language frontend #1249
Merged
New Go language frontend #1249
+3,630
−3,609
Conversation
This file contains 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
oxisto
requested review from
KuechA,
maximiliankaul and
konradweiss
as code owners
oxisto
force-pushed
the
new-go-frontend
branch
4 times, most recently
from
2f9dd7a to
0ff8e58
Compare
oxisto
force-pushed
the
type-system-overhaul
branch
2 times, most recently
from
6fd7e5f to
ba0de3f
Compare
oxisto
force-pushed
the
type-system-overhaul
branch
from
ba0de3f to
340a5f8
Compare
Closed
oxisto
force-pushed
the
type-system-overhaul
branch
from
340a5f8 to
5338ea5
Compare
oxisto
force-pushed
the
type-system-overhaul
branch
2 times, most recently
from
0c5239f to
023a53d
Compare
oxisto
force-pushed
the
new-go-frontend
branch
2 times, most recently
from
77e4a4b to
6687080
Compare
oxisto
force-pushed
the
new-go-frontend
branch
5 times, most recently
from
79fbf41 to
51ebb46
Compare
This PR contains a complete re-write of the Go language frontend based on JNA instead of a pure Go implementation. While this has a slight performance drawback (I am working on providing some numbers), it has infinitly better debugging capabilities. The handlers and most of the frontend itself is written in Kotlin using our classes and we than use a JNA bridge to a C API wrapper around the Go `go/ast` package. Depends on #1199 Fixes #1256 Fixes #1257 Fixes #1258 Fixes #1259
|
Kudos, SonarCloud Quality Gate passed! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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 PR contains a complete re-write of the Go language frontend based on JNA instead of a pure Go implementation. While this has a slight performance drawback (I am working on providing some numbers), it has infinitly better debugging capabilities. The handlers and most of the frontend itself is written in Kotlin using our classes and we than use a JNA bridge to a C API wrapper around the Go
go/astpackage.Depends on #1199
Fixes #1256
Fixes #1257
Fixes #1258
Fixes #1259