RFR: 8357551: RISC-V: support CMoveF/D vectorization [v2]
Hamlin Li
mli at openjdk.org
Mon Nov 17 10:27:06 UTC 2025
On Mon, 17 Nov 2025 09:51:39 GMT, Emanuel Peter <epeter at openjdk.org> wrote:
>> In this PR, no, this one will only implement CMoveF/D and enable the vectorization of CMoveF/D, so do some preparation for https://github.com/openjdk/jdk/pull/28231.
>> To guarantee the generation of CMoveI/L, seems to me we need to improve the cost model when transfrom a phi to a conditional move. I can have a invetigation later, as this impact how & whether CMoveL/I can be generated and be vectorized accordingly. File https://bugs.openjdk.org/browse/JDK-8371984 to track it.
>
> Ok. Sounds good. Just note: getting the cost model right here can be really difficult. People have played with the cost model in recent years, and it has also led to regressions in some cases. Just FYI, I'm not stopping you from trying if you like ;)
Thanks for reminding! :) That's also the reason I won't do it in this PR.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28309#discussion_r2533522249
More information about the hotspot-dev
mailing list