RFR: 8352607: RISC-V: use cmove in min/max when Zicond is supported

Fei Yang fyang at openjdk.org
Tue Mar 25 06:44:12 UTC 2025


On Mon, 24 Mar 2025 16:19:24 GMT, Vixea <duke at openjdk.org> wrote:

>> I'm pretty sure the p550(hifive primier p550) doesn't support zicond but does support zbb, zba
>
> Umm anyway the p550(hifive primier and megrez don't support zicond but do support zbb/zba.
> https://github.com/llvm/llvm-project/commit/5d03235c73476dfa3d2dd48c76de106fd1aa2ac7
> Oh heh I got the question backwards then in that case idk too.

> But, anyway in fact you can just consider this as a code cleanup, in this sense seems it should be good?

Seems fine to me in that respect. We can fix the cost in another PR.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24153#discussion_r2011417580


More information about the hotspot-compiler-dev mailing list