Skip to content

[benchmark] Small Driver Tweaks #20826

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

Merged
merged 1 commit into from
Nov 28, 2018
Merged

Conversation

palimondo
Copy link
Contributor

@palimondo palimondo commented Nov 28, 2018

Equalize reported memory usage with & without --verbose

@palimondo palimondo requested a review from eeckstein November 28, 2018 15:09
@nordicio

This comment has been minimized.

@palimondo

This comment has been minimized.

The use of `--verbose` parameter was affecting the reported memory usage (`--memory`), because it front-loads initialization of string interpolation and printing.

By always computing the configuration string and always calling print, the baseline memory measurement no longer includes this constant overhead.
@palimondo
Copy link
Contributor Author

@eeckstein Please review 🙏

Copy link
Contributor

@eeckstein eeckstein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@eeckstein
Copy link
Contributor

@swift-ci benchmark

@eeckstein
Copy link
Contributor

@swift-ci smoke test

1 similar comment
@eeckstein
Copy link
Contributor

@swift-ci smoke test

@eeckstein
Copy link
Contributor

@swift-ci benchmark

@swift-ci
Copy link
Contributor

Build comment file:

No performance and code size changes


@palimondo palimondo merged commit a05e6bc into swiftlang:master Nov 28, 2018
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.

4 participants