Skip to content

Conversation

@bitfaster
Copy link
Owner

This appears to have little effect on the result, but is best practice as mentioned here.

@bitfaster bitfaster changed the title benchmark avoid dead code elimination benchmarks avoid dead code elimination Nov 18, 2024
@coveralls
Copy link

coveralls commented Nov 19, 2024

Coverage Status

coverage: 99.217%. remained the same
when pulling 8693304 on users/alexpeck/benchfix
into 1e7011e on main.

@bitfaster bitfaster merged commit 6de8fa6 into main Nov 19, 2024
13 checks passed
@bitfaster bitfaster deleted the users/alexpeck/benchfix branch November 19, 2024 04:26
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