On Wed, 23 Apr 2025 08:54:23 GMT, Hamlin Li <m...@openjdk.org> wrote:

>> The intention is to align `_features_string` with `_features` which 
>> enumerates well-known CPU capabilities JVM manages. As of now, 
>> `_features_string` contains more information, so I introduced 
>> `_cpu_info_string` to keep it.
>> 
>> Speaking of `test/lib/jdk/test/whitebox/cpuinfo/CPUInfo.java`, the approach 
>> chosen there may be fine for a test library, but we need a more stable API 
>> between JVM and JDK.
>
> I'm fine with this.
> But it might be better to change the spliting regex of  `_features_string` in 
> CPUFeatures.java to support riscv cpu features format.

Ok, I pushed an update. Let me know what you think about it.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24462#discussion_r2057061162

Reply via email to