On Mon, 17 Nov 2025 09:51:39 GMT, Emanuel Peter <[email protected]> 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
