[go: nahoru, domu]

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

Adding/updating crypto benchmarks #9

Merged
merged 1 commit into from
Aug 20, 2018
Merged

Conversation

jerrylui803
Copy link
@jerrylui803 jerrylui803 commented Jul 30, 2018

Signed-off-by: Jerry Lui jerrylui803@gmail.com

@sehirst
Copy link
sehirst commented Jul 30, 2018

Jerry works with me and we worked together on this. Please add me as a reviewer for the PR.

@karianna karianna added the enhancement New feature or request label Jul 30, 2018
@sehirst
Copy link
sehirst commented Jul 30, 2018

@jerrylui803 Please add a bit more description in your commit message. There should be a subject followed by a blank line, and then the rest of the description. Just enough to explain why this commit is happening and what the code is used for. A few sentences should be enough.

@karianna
Copy link
Contributor

@sehirst - I've sent you an invite to the testing team

This commit adds crypto benchmarks that have been used in
bumblebench but were not commited before it was open sourced.
Also some crypto benchmarks are updated with more recent code.

Signed-off-by: Jerry Lui jerrylui803@gmail.com
@sehirst
Copy link
sehirst commented Aug 20, 2018

This change is updating some existing crypto perf benchmarks and adding others that we have been using for a while now with an outdated version of bumblebench. Changes are LGTM.

@sehirst sehirst changed the title Adding crypto test bench Adding/updating crypto benchmarks Aug 20, 2018
@sehirst sehirst merged commit d409bf9 into adoptium:master Aug 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants