Skip to content

[RVV] add rvv f32 kernel for ppmm#9962

Open
velonica0 wants to merge 3 commits intogoogle:masterfrom
velonica0:rvv-ppmm
Open

[RVV] add rvv f32 kernel for ppmm#9962
velonica0 wants to merge 3 commits intogoogle:masterfrom
velonica0:rvv-ppmm

Conversation

@velonica0
Copy link
Copy Markdown
Contributor

Part of #9954

Add RVV kernel for f32-ppmm with MR={1,2,4} and NR={m1,m2,m4} variants. Tested on SpacemiT K1 CPU and K3 CPU, both VLEN=256.

Comment thread cmake/gen/rvv_microkernels.cmake Outdated
src/x32-transposec/gen/x32-transposec-8x8-rvv.c
src/x32-transposec/gen/x32-transposec-16x8-rvv.c
src/x32-transposec/gen/x32-transposec-32x8-rvv.c)
src/x32-transposec/gen/x32-transposec-32x8-rvv.c
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

In all 3 PRs, there will be merge conflicts because these are all added in the same place. Keeping these lists alphabetized will avoid this.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thank you for your review, I have made the changes.

Add RVV kernel for f32-ppmm with MR={1,2,4} and NR={m1,m2,m4} variants.
Tested on SpacemiT K1 CPU and K3 CPU, both VLEN=256.
Move f32-ppmm entries from end of PROD/NON_PROD lists to alphabetical
position (between f32-maxpool and f32-qs8-vcvt) to avoid merge conflicts
with other RVV PRs.
@velonica0
Copy link
Copy Markdown
Contributor Author

Hi, @dsharlet
My apologies. Fixed a CI error caused by a missing ). It should be fine now for 3 PRs(#9963 #9964).
Sorry for that.

@velonica0
Copy link
Copy Markdown
Contributor Author

velonica0 commented Apr 23, 2026

Hi, @dsharlet
Can these be merged?

@velonica0
Copy link
Copy Markdown
Contributor Author

Hi, @dsharlet My apologies. Fixed a CI error caused by a missing ). It should be fine now for 3 PRs(#9963 #9964). Sorry for that.

Hi, @dsharlet
Can these be merged? I see CI is pass!

@velonica0
Copy link
Copy Markdown
Contributor Author

ping @dsharlet
Thanks!

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.

2 participants