-
Notifications
You must be signed in to change notification settings - Fork 14
Typer + Inferencer need to lose some weight #501
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
Comments
I estimate this can reduce allocation by 3-4% overall. |
Here's the followup work for Context: https://github.com/retronym/scala/tree/topic/post-typer-context |
- Move the checkDead module out of Typer - Use a new bit in ContextMode to track when to suppress the warning, rather than maintaining the stack of the symbols of enclosing applications - Only do any of this when under -Ywarn-dead code References scala/scala-dev#501
- Move the checkDead module out of Typer - Use a new bit in ContextMode to track when to suppress the warning, rather than maintaining the stack of the symbols of enclosing applications - Only do any of this when under -Ywarn-dead code References scala/scala-dev#501
- Move the checkDead module out of Typer - Use a new bit in ContextMode to track when to suppress the warning, rather than maintaining the stack of the symbols of enclosing applications - Only do any of this when under -Ywarn-dead code References scala/scala-dev#501 (cherry picked from commit fc72bbb)
Looks like there were a few PRs addressing this, but not all of it? Jason describes in scala/scala#6616:
Which looks like are still TODO. |
Uh oh!
There was an error while loading. Please reload this page.
Before:
After faster/typer-diet
Yep, 24 bytes rather than the hulking 104 we're currently hauling around.
As a bonus, we can avoid allocating
Typer
-s altogether inTypingTransformers
.We'll need a followup ticket to give
Context
the same treatment, its still a monster.The text was updated successfully, but these errors were encountered: