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

Reduce the number of calls to git to get source information #275

Merged
merged 1 commit into from
Nov 17, 2024

Conversation

smarr
Copy link
Owner

@smarr smarr commented Nov 17, 2024

There’s a notable delay when ReBench starts, and this might be one source of it.

Instead of doing repeated calls to git show, pack everything into one call and use \0 as delimiter.

Unfortunately, the real performance issue may be in cpuinfo, which takes quite a bit of time.

There’s a notable delay when ReBench starts, and this might be one source of it.

Instead of doing repeated calls to `git show`, pack everything into one call and use \0 as delimiter.

Signed-off-by: Stefan Marr <[email protected]>
@smarr smarr merged commit 5d61eee into master Nov 17, 2024
10 of 11 checks passed
@smarr smarr deleted the optimize_determine_source branch November 17, 2024 21:15
@coveralls
Copy link

Coverage Status

coverage: 53.805% (-0.005%) from 53.81%
when pulling 1bf94fa on optimize_determine_source
into 9a95df7 on master.

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

Successfully merging this pull request may close these issues.

2 participants