Skip to content
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

Remove support for benchmarking with the Tier 2 interpreter #357

Open
mdboom opened this issue Feb 11, 2025 · 0 comments
Open

Remove support for benchmarking with the Tier 2 interpreter #357

mdboom opened this issue Feb 11, 2025 · 0 comments

Comments

@mdboom
Copy link
Contributor

mdboom commented Feb 11, 2025

We haven't been optimizing the tier 2 interpreter (without the JIT) for performance in a long time. We haven't included it in the weekly benchmarks for at least a year.

I think we should remove the option to benchmark with the tier 2 interpreter, because it would free up the limited space of 10 options for other things. I just added a box for tail calling, putting us at the maximum number of options.

@markshannon: Is this ok for you, as the biggest user of the tier 2 interpreter?

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

No branches or pull requests

1 participant