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

[cases] add rvv_bench #570

Closed
wants to merge 1 commit into from
Closed

[cases] add rvv_bench #570

wants to merge 1 commit into from

Conversation

SharzyL
Copy link
Contributor

@SharzyL SharzyL commented May 10, 2024

No description provided.

@sequencer
Copy link
Member

Thanks @SharzyL for adding @camel-cdr tests here!
These benchs will be the guide for T1 performance tuning.
T1 will use it for CI in the future, after T1 successfully passed all workloads provided by @camel-cdr, we will add that repo as dependency.

Copy link
Member

@sequencer sequencer left a comment

Choose a reason for hiding this comment

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

Please add corresponding tests to CI

@SharzyL SharzyL force-pushed the cases_bench branch 2 times, most recently from a7968f5 to 7df667a Compare May 14, 2024 02:16
@qinjun-li qinjun-li force-pushed the cases_bench branch 3 times, most recently from d70adc4 to 7df667a Compare May 14, 2024 08:52
@SharzyL SharzyL force-pushed the cases_bench branch 2 times, most recently from ac512c8 to 918fdf9 Compare May 14, 2024 09:11
"rvv_bench.ascii_to_utf16": 1541742,
"rvv_bench.ascii_to_utf32": 673956,
"rvv_bench.byteswap": 3456435,
"rvv_bench.chacha20": 0,
Copy link
Member

Choose a reason for hiding this comment

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

cc @camel-cdr, it seems chacha20 and poly1305 doesn't go to auto vectorization?

Choose a reason for hiding this comment

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

I didn't explicitly disable it, but it doesn't get autovectorized by todays compilers.

@sequencer
Copy link
Member

why CI doesn't work...

@SharzyL SharzyL force-pushed the cases_bench branch 2 times, most recently from 1c0a1e4 to 4b2279f Compare May 20, 2024 16:10
@sequencer
Copy link
Member

why CI doesn't work...

@sequencer sequencer closed this May 20, 2024
@sequencer sequencer reopened this May 20, 2024
@sequencer sequencer closed this May 20, 2024
@sequencer sequencer reopened this May 20, 2024
@sequencer
Copy link
Member

@SharzyL please reopen it, github failed to recover:(

@sequencer sequencer closed this May 20, 2024
@SharzyL SharzyL mentioned this pull request May 21, 2024
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.

3 participants