RFR: 8322753: RISC-V: C2 ReverseBytesV [v2]

Hamlin Li mli at openjdk.org
Wed May 8 10:42:11 UTC 2024


On Wed, 8 May 2024 10:30:45 GMT, Ludovic Henry <luhenry at openjdk.org> wrote:

>> Hamlin Li has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains seven commits:
>> 
>>  - minor fix
>>  - merge master
>>  - remove reverse bits
>>  - fix test filter
>>  - fix zvbb flag; fix tests
>>  - merge master
>>  - ReverseV/ReverseBytesV: Initial Commit
>
> test/hotspot/jtreg/compiler/vectorapi/VectorReverseBytesTest.java line 46:
> 
>> 44:  * @requires vm.compiler2.enabled
>> 45:  * @requires (os.simpleArch == "x64" & vm.cpu.features ~= ".*avx2.*") |
>> 46:  *           os.arch == "aarch64" |
> 
> You can avoid the newline here

> That is still leftover from merging master.

Thanks for catching, fixed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19120#discussion_r1593819899


More information about the hotspot-dev mailing list