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