RFR: 8312425: [vectorapi] AArch64: Optimize vector math operations with SLEEF [v3]

Hamlin Li mli at openjdk.org
Thu Apr 11 10:45:46 UTC 2024


On Thu, 11 Apr 2024 10:36:03 GMT, Hamlin Li <mli at openjdk.org> wrote:

>> Hi,
>> Can you help to review the patch?
>> This pr is based on previous work and discussion in [pr 16234](https://github.com/openjdk/jdk/pull/16234), [pr 18294](https://github.com/openjdk/jdk/pull/18294).
>> 
>> Compared with previous prs, the major change in this pr is to integrate the source of sleef (for the steps, please check `src/jdk.incubator.vector/linux/native/libvectormath/README`), rather than depends on external sleef things (header or lib) at build or run time.
>> Besides of this change, also modify the previous changes accordingly, e.g. remove some uncessary files or changes especially in make dir of jdk.
>> 
>> Besides of the code changes, one important task is to handle the legal process.
>> 
>> Thanks!
>
> Hamlin Li has updated the pull request incrementally with one additional commit since the last revision:
> 
>   fix performance issue

Thanks everyone for discussion about the direction (integrate source or lib).

We did have some implementation for integrating sleef lib into jdk, but seems previously the most strong opinion is to integrate the sleef source into jdk.
I know there are cons and pros for every solution, but I will stick to current solution unless everyone can reach another agreement.

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

PR Comment: https://git.openjdk.org/jdk/pull/18605#issuecomment-2049410731


More information about the build-dev mailing list