Skip to content

Avoid some instances of boxing - #26939

Draft
SolalPirelli wants to merge 4 commits into
scala:mainfrom
dotty-staging:solal/nobox
Draft

Avoid some instances of boxing#26939
SolalPirelli wants to merge 4 commits into
scala:mainfrom
dotty-staging:solal/nobox

Conversation

@SolalPirelli

@SolalPirelli SolalPirelli commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Notes for myself when I come back from vacation:

  • caffc6e breaks compat; worth partial revert, but keep the NameTable removal as it's really leaky anyway
  • I think I forgot to push one commit (the one with hashing products for namekinds)

Looking at a helloWorld flame graph for 100 iterations at a time, I noticed 1.5% was in integer boxing, so decided to look further.

Self-contained commits, probably best to review commit-by-commit.

Have you relied on LLM-based tools in this contribution?

No

How was the solution tested?

Covered by existing tests (this is a refactoring)

@dottybot

Copy link
Copy Markdown
Member

Benchmarks started. Workflow run.

@dottybot

Copy link
Copy Markdown
Member

Benchmarks completed. Overview.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants