-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
adding GenBCode to dotc #9
Conversation
LGTM. I'll merge it into the backend branch. Are you working on the dotc backend currently? Just so that we don't work on the same thing. |
@lrytz The progress I've made on "GenBCode in dotc" is this PR. Before making further progress I have questions about the dotc-way to types and symbols. It would be great if you had time to discuss that. |
I'll be at EPFL tomorrow. Should we meet around 13:30? Maybe @odersky can join to answer the questions, I'll take notes. |
@lrytz Let's start tomorrow by reviewing the pending TODOs, and then we can ask for help :) |
doc(match-type): fix type lambda syntax
Added preprocessor which replaces logo in docs. Provide basic Scala logo
This PR integrates into dotc the main classes making up GenBCode, including their adaptation to Context passing.
What remains to be done are method-level changes. Examples of changes I'll work on next:
ArrayValue
, replacementJavaSeqLiteral
LabelDef
yet leave in place all infrastructure for emitting jumps (still needed for try-catch-finally and others)Other changes (hint: that can be performed in parallel) are about replacing usages of the scalac Symbol and Type APIs with their dotc counterparts.
review by @lrytz