-
Notifications
You must be signed in to change notification settings - Fork 344
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
Add RVV qs8/qu8 gavgpool kernels, configs and tests. #7031
base: master
Are you sure you want to change the base?
Conversation
cmake/gen/rvv_microkernels.cmake
Outdated
@@ -44,8 +44,12 @@ SET(PROD_RVV_MICROKERNEL_SRCS | |||
src/f32-vlrelu/gen/f32-vlrelu-rvv-u4v.c | |||
src/f32-vrelu/gen/f32-vrelu-rvv-u4v.c | |||
src/f32-vrsqrt/gen/f32-vrsqrt-rvv-rsqrt-u4v.c | |||
src/qs8-gavgpool/gen/qs8-gavgpool-7p7x-minmax-fp32-rvv-c2v.c |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
c2v function/file naming inconsistent with rvv kernels
should be u2v meaning unrolled to 2 vectors.. aka m2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
f32 gavgpool uses the old style naming as well, should get fixed
-- bc650ef by Mihai.Olinovici <[email protected]>: Add qs8/qu8 gavgpool kernels, configs and tests. -- b0cfeb2 by Mihai.Olinovici <[email protected]>: Adhere to standard RVV naming. -- 8c941c0 by Mihai.Olinovici <[email protected]>: Remove old unipass reminants. FUTURE_COPYBARA_INTEGRATE_REVIEW=#7031 from imaginationtech:img_patch25_qs8_gavgpool 8c941c0 PiperOrigin-RevId: 672712922
-- bc650ef by Mihai.Olinovici <[email protected]>: Add qs8/qu8 gavgpool kernels, configs and tests. -- b0cfeb2 by Mihai.Olinovici <[email protected]>: Adhere to standard RVV naming. -- 8c941c0 by Mihai.Olinovici <[email protected]>: Remove old unipass reminants. FUTURE_COPYBARA_INTEGRATE_REVIEW=#7031 from imaginationtech:img_patch25_qs8_gavgpool 8c941c0 PiperOrigin-RevId: 672712922
-- bc650ef by Mihai.Olinovici <[email protected]>: Add qs8/qu8 gavgpool kernels, configs and tests. -- b0cfeb2 by Mihai.Olinovici <[email protected]>: Adhere to standard RVV naming. -- 8c941c0 by Mihai.Olinovici <[email protected]>: Remove old unipass reminants. FUTURE_COPYBARA_INTEGRATE_REVIEW=#7031 from imaginationtech:img_patch25_qs8_gavgpool 8c941c0 PiperOrigin-RevId: 672712922
We are currently working on deprecating gavgpool and replacing it with static_mean, which is a generalization of gavgpool. I suggest directing your efforts towards implementing rsum and rdsum (which implement static_mean) rather than gavgpool. |
-- bc650ef by Mihai.Olinovici <[email protected]>: Add qs8/qu8 gavgpool kernels, configs and tests. -- b0cfeb2 by Mihai.Olinovici <[email protected]>: Adhere to standard RVV naming. -- 8c941c0 by Mihai.Olinovici <[email protected]>: Remove old unipass reminants. -- 8304b4d by Mihai.Olinovici <[email protected]>: Regenerate make files. FUTURE_COPYBARA_INTEGRATE_REVIEW=#7031 from imaginationtech:img_patch25_qs8_gavgpool 8304b4d PiperOrigin-RevId: 672712922
No description provided.