RFR: 8343600: RISC-V: enable CRC32 intrinsic when either Zba or RVV are supported

[Hamlin Li]

On Wed, 6 Nov 2024 09:42:27 GMT, Fei Yang <fyang at openjdk.org> wrote:

> Yeah, kind of. I guess that issue is also there even we have Zba. But I don't see a good way to make it size aware. Maybe we should only keep the vector version?

I'm not sure. I saw the trend when that pr was in progress.
Although I guess the big size of data is one of main usage cases for CRC, but I don't know how big it can get.
So, for now I would suggest to keep both Zba and RVV version at the same time, and in the future if we get regression report in the real world use case, we can just turn off CRC intrinsic when RVV is not supported. How do you think about it?

-------------

PR Comment: https://git.openjdk.org/jdk/pull/21897#issuecomment-2459158137