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