RFR: 8320646: RISC-V: C2 VectorCastHF2F [v7]

[Hamlin Li]

On Fri, 8 Mar 2024 07:50:29 GMT, Fei Yang <fyang at openjdk.org> wrote:

>> Thanks for testing!
>
>> Seems not necessary currently, unless in the future there is other extension starting with `zfh`, e.g. `zfhX`, but seems that chance is low IMHO. And, I think it might not work for the last extension in the CPU feature string if it's with a comma (as the cpu feature string is converted from a list to string, so the last one will not ending with a`,` ? ), so for new and long named extensions, it should be without a comma.
> 
> Yeah, that makes sense to me. I think we should remove this comma from conditions `(os.arch == "riscv64" & vm.cpu.features ~= ".*zfh,.*")` on jdk master. Can you fix that? Then we will have a more simpler rule when writing those conditions. That is, for single-character extensions like `c` or `v`, we need to add this comma. But for multi-character extensions like `zfh` or `zvfh`, we should not have it to avoid the issue you mentioned.

Sure, I created https://bugs.openjdk.org/browse/JDK-8327689 to track it.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17698#discussion_r1517626629