RFR: 8353786: Migrate Vector API math library support to FFM API [v3]
Vladimir Ivanov
vlivanov at openjdk.org
Mon Apr 7 23:25:26 UTC 2025
On Mon, 7 Apr 2025 17:44:33 GMT, Chen Liang <liach at openjdk.org> wrote:
>> Vladimir Ivanov has updated the pull request incrementally with one additional commit since the last revision:
>>
>> features_string -> cpu_info_string
>
> src/jdk.incubator.vector/share/classes/jdk/incubator/vector/CPUFeatures.java line 44:
>
>> 42: String featuresString = VectorSupport.getCPUFeatures();
>> 43: debug(featuresString);
>> 44: String[] features = featuresString.toLowerCase().split(", "); // ", " is used as a delimiter
>
> Please use `toLowerCase(Locale.ROOT)`: if the system locale is turkish, `I` and dotless i are two letters, and the dotless i will fail in the subsequent `validateFeatures` assertion. Same for `hasFeature`.
Good point. Fixed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24462#discussion_r2032135321
More information about the core-libs-dev
mailing list