RFR: 8353786: Migrate Vector API math library support to FFM API [v10]

Vladimir Ivanov vlivanov at openjdk.org
Wed Apr 23 00:40:07 UTC 2025


On Tue, 22 Apr 2025 14:46:21 GMT, Ludovic Henry <luhenry at openjdk.org> wrote:

>> Vladimir Ivanov has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 24 additional commits since the last revision:
>> 
>>  - Merge remote-tracking branch 'origin/master' into vector.math.01.java
>>  - RVV and SVE adjustments
>>  - fix broken merge
>>  - Merge branch 'master' into vector.math.01.java
>>  - Fix debugName handling
>>  - Merge branch 'master' into vector.math.01.java
>>  - RVV and SVE adjustments
>>  - Merge branch 'master' into vector.math.01.java
>>  - Fix windows-aarch64 build failure
>>  - features_string -> cpu_info_string
>>  - ... and 14 more: https://git.openjdk.org/jdk/compare/41f2363b...88eacc48
>
> src/jdk.incubator.vector/share/classes/jdk/incubator/vector/VectorMathLibrary.java line 75:
> 
>> 73:             return switch (StaticProperty.osArch()) {
>> 74:                 case "amd64", "x86_64" -> SVML;
>> 75:                 case "aarch64" -> SLEEF;
> 
> We should be supporting SLEEF on `riscv64`. Was there a specific motivation not to include it here?

Goot catch, fixed.

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

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


More information about the hotspot-compiler-dev mailing list