RFR: 8320995: RISC-V: C2 PopCountVI [v4]

Hamlin Li mli at openjdk.org
Wed May 8 08:53:22 UTC 2024


On Wed, 8 May 2024 00:06:23 GMT, Fei Yang <fyang at openjdk.org> wrote:

>> Hamlin Li has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   minor fix
>
> test/hotspot/jtreg/compiler/vectorization/TestPopCountVectorLong.java line 30:
> 
>> 28: * @requires ((os.arch=="x86" | os.arch=="i386" | os.arch=="amd64" | os.arch=="x86_64") & vm.cpu.features ~= ".*avx512bw.*") |
>> 29: *           os.simpleArch == "aarch64" |
>> 30: *           (os.arch == "riscv64" & vm.cpu.features ~= ".*zvbb,.*")
> 
> Suggestion: `(os.arch == "riscv64" & vm.cpu.features ~= ".*zvbb.*")`
> The comma should not be there. See: https://bugs.openjdk.org/browse/JDK-8327689

Thanks, fixed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19065#discussion_r1593664140


More information about the hotspot-dev mailing list