Skip to content

Conversation

@lhpqaq
Copy link
Contributor

@lhpqaq lhpqaq commented Nov 20, 2024

I am working to gradually add support for RISC-V Vector extension. Does the main branch have this plan?
I can follow your suggestion, such as changing to integer calculation like neon

@ram-mohan
Copy link
Contributor

I am working to gradually add support for RISC-V Vector extension. Does the main branch have this plan? I can follow your suggestion, such as changing to integer calculation like neon

Thank you for your interest. We are not working on this path currently. some practices I follow, generally during our development we try to make changes such that they work on both 32 and 64 bit variants. As integer operations tend to be faster than float arithmetic, while developing intrinsics, we chose fixed point implementation. We ensure the changes are clang-formatted so that the code style is uniform across files and sufficient testing needs to be done to ensure nothing trivial breaks.

@lhpqaq lhpqaq marked this pull request as draft December 2, 2024 02:33
@lhpqaq
Copy link
Contributor Author

lhpqaq commented Dec 2, 2024

I am working to gradually add support for RISC-V Vector extension. Does the main branch have this plan? I can follow your suggestion, such as changing to integer calculation like neon

Thank you for your interest. We are not working on this path currently. some practices I follow, generally during our development we try to make changes such that they work on both 32 and 64 bit variants. As integer operations tend to be faster than float arithmetic, while developing intrinsics, we chose fixed point implementation. We ensure the changes are clang-formatted so that the code style is uniform across files and sufficient testing needs to be done to ensure nothing trivial breaks.

Hi, I have implemented another version of RVV in #334, and I will continue to work on this PR in the future. Looking forward to your guidance

@ram-mohan
Copy link
Contributor

if this is no longer required, can this be closed?

@lhpqaq lhpqaq closed this Dec 4, 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.

2 participants