On Fri, 30 Aug 2024 16:57:18 GMT, Magnus Ihse Bursie <i...@openjdk.org> wrote:
>> [JDK-8312425](https://bugs.openjdk.org/browse/JDK-8312425) is looking to >> optimize vector math operations by leveraging the SLEEF library. For legal >> reasons the actual contribution of the SLEEF files needs to be handled >> separately. >> >> This is a new attempt at solving >> [JDK-8329816](https://bugs.openjdk.org/browse/JDK-8329816); the original >> attempt is here: https://github.com/openjdk/jdk/pull/19185. This PR is based >> on the discussions on how to move forward that was held in that original PR. > > Magnus Ihse Bursie has updated the pull request incrementally with two > additional commits since the last revision: > > - Use "whitespace" as an uncountable noun > > Co-authored-by: Erik Joelsson <37597443+erik...@users.noreply.github.com> > - I suck at English verb forms > > Co-authored-by: Erik Joelsson <37597443+erik...@users.noreply.github.com> I've applied the generated sleef headers to aarch64/riscv64 patches (`Optimize vector math operations with SLEEF`), all good (the so files are built successfully, the tests jdk/incubator/vector/(Double|Float)MaxVectorTests.java run successfully, also output log is correct), although I don't run jmh tests, I think it's fine. Thanks @magicus @erikj79 for the efficient co-work! ------------- Marked as reviewed by mli (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/20781#pullrequestreview-2273399318