-
Notifications
You must be signed in to change notification settings - Fork 3.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
Assortiment of cleanups and comments around the backend #3826
Conversation
Tests went fine, review by @retronym |
Thanks @retronym for insisting on a before-after comparison: I found a serious issue and have to give up on introducing For example, the tree transformation in the lambdalift phase, which lifts definitions (and modifies their symbol's owner), is executed I'll go back to just save the |
The final flag is computed correctly by javaFlags.
removed |
LGTM. In the future please emphasize the why over what when writing commit messages. A year from now it's going to be a lot easier to work out what has been changed in the diff than why it was done. I'd like to emphasize this point in our policy: https://github.com/scala/scala/wiki/Pull-Request-Policy |
Assortiment of cleanups and comments around the backend
No description provided.