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

[Vladimir Ivanov]

On Mon, 7 Apr 2025 17:01:19 GMT, Vladimir Kozlov <kvn 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 60:
> 
>> 58:     }
>> 59: 
>> 60:     public static class X64 {
> 
> Should we create `src/jdk.incubator.vector/cpu/` for CPU specific information? As separate refactoring.

To clarify: are you suggesting to move platform-specific classes into a separate package or platform-specific location? 

It does make sense to separate platform-specific parts into their own classes once amount of code grows over some limit. For now it doesn't look too attractive since amount of code is very small.

> src/jdk.incubator.vector/share/classes/jdk/incubator/vector/VectorMathLibrary.java line 100:
> 
>> 98: 
>> 99:     /**
>> 100:      * Naming convention in SVML vector math library.
> 
> Does this library has code for all AVX configurations?

Yes, there are 4 configurations (`-XX:UseAVX=[0..3]`) in total covered by SVML library.

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

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