RFR: 8357551: RISC-V: support CMoveF/D vectorization [v7]

Emanuel Peter epeter at openjdk.org
Mon Dec 8 13:45:10 UTC 2025


On Mon, 8 Dec 2025 13:39:23 GMT, Hamlin Li <mli at openjdk.org> wrote:

>> Nice work! I did not review the RISC-V specific changes, but had a look at the tests. Wow, there are a lot of them, and that's a good thing :)
>> 
>> I have a few comments below, to consider for improvement.
>
>> Nice work! I did not review the RISC-V specific changes, but had a look at the tests. Wow, there are a lot of them, and that's a good thing :)
>> 
>> I have a few comments below, to consider for improvement.
> 
> @eme64 Thanks for having a look.
> Seems there is a race condition here. :) I just triggered the integration. 
> I'll file new bug to address your comment.

@Hamlin-Li Sounds good. I don't blame you, you had 2 reviewers ;)
I also did not run internal testing. Most likely everything will be fine.
Thanks for addressing my comments in the future, much appreciated!

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

PR Comment: https://git.openjdk.org/jdk/pull/28309#issuecomment-3627007431


More information about the core-libs-dev mailing list