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

Hamlin Li mli at openjdk.org
Mon Dec 8 18:01:22 UTC 2025


On Mon, 8 Dec 2025 13:34:10 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> test/hotspot/jtreg/compiler/c2/irTests/TestConditionalMove.java line 39:
>> 
>>> 37:  * @summary Auto-vectorization enhancement to support vector conditional move.
>>> 38:  * @library /test/lib /
>>> 39:  * @run driver compiler.c2.irTests.TestConditionalMove
>> 
>> Suggestion:
>> 
>>  * @run driver ${test.main.class}
>> 
>> Might as well do that now. Avoids wrong copy of class name, which can lead to wrong test being run.
>
> Also: if you are already renaming these tests, you might move them to a better directory as well. We want to avoid the `irTests` directory in the future,  and sort by topic instead.
> 
> Idea: put it under a `compiler/c2/cmove` directory.

addressed in https://github.com/openjdk/jdk/pull/28702.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28309#discussion_r2599583943


More information about the core-libs-dev mailing list