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

Ludovic Henry luhenry at openjdk.org
Wed May 8 10:33:55 UTC 2024


On Wed, 8 May 2024 10:21:18 GMT, Hamlin Li <mli at openjdk.org> wrote:

>> Hi,
>> Can you review this patch to add ReverseBytesV intrinsic?
>> Thanks.
>
> 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

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

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


More information about the hotspot-dev mailing list