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

Safe optimizations #45

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Safe optimizations #45

wants to merge 2 commits into from

Conversation

wingo
Copy link
Contributor

@wingo wingo commented Aug 28, 2019

Change Chez, Gambit, and Gerbil to avoid unsafe optimizations. Fixes #43.

@ghost
Copy link

ghost commented Sep 27, 2019

If we're heading that way, then you might also want to change https://github.com/ecraven/r7rs-benchmarks/blob/master/bench#L760.

${GERBIL} -d $(dirname "$1") -exe -static -O -prelude "(declare (not safe))" -o "${1%.scm}.exe" "$1"

@belmarca
Copy link

Why not just have a safe and unsafe benchmarks?

Maybe a "defaults" benchmark?

@svenha
Copy link
Contributor

svenha commented Nov 12, 2019

Producing benchmark results in safe mode and in unsafe mode will help a lot of Schemers.

A defaults run could help beginners, but beginners should not benchmark too much :-) So, I would avoid a defaults run.

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.

Avoid unsafe optimizations
3 participants