On Thu, 27 Jun 2024 11:53:38 GMT, Fei Gao <f...@openjdk.org> wrote: >> in progress... > > Hi @Hamlin-Li , thanks for your work. > > I tried to run benchmarks, > [FloatMaxVector](https://github.com/openjdk/panama-vector/blob/vectorIntrinsics/test/micro/org/openjdk/bench/jdk/incubator/vector/operation/FloatMaxVector.java#L1068) > and > [DoubleMaxVector](https://github.com/openjdk/panama-vector/blob/vectorIntrinsics/test/micro/org/openjdk/bench/jdk/incubator/vector/operation/DoubleMaxVector.java#L1068), > on different aarch64 machines. > > Here is the data I got for `TANH`, with args `-i 5 -f 3 -wi 3 -foe true > -jvmArgs -Xms4g -Xmx4g -XX:+AlwaysPreTouch -XX:ObjectAlignmentInBytes=16`: > > > // NEON machine > Benchmark (size) Mode Cnt Units Perf gain > DoubleMaxVector.TANH 1024 thrpt 15 ops/ms -38% > FloatMaxVector.TANH 1024 thrpt 15 ops/ms -26% > > > > // 128-bit sve machine (TANH also implemented with NEON) > Benchmark (size) Mode Cnt Units Perf gain > DoubleMaxVector.TANH 1024 thrpt 15 ops/ms -19% > FloatMaxVector.TANH 1024 thrpt 15 ops/ms ~00% > > > The performance of vector stubs for `TANH` looks not quite stable on > different NEON machines. Since this pr does not provide `TANH` interface on > sve machines for [the performance > regression](https://github.com/openjdk/jdk/pull/16234/commits/2a7730d6acbac80438a43d1502cff6a476f8b5b5#diff-9112056f732229b18fec48fb0b20a3fe824de49d0abd41fbdb4202cfe70ad114R8521-R8525), > how about also disabling it on NEON for the same reason? WDYT? > > Thanks.
@fg1417 Thanks for testing. Sure, I can do that based on your test result, I will restart work on it after https://github.com/openjdk/jdk/pull/19185 is integrated. @theRealAph I lost my previous vm, so currently I only generate the header files, but did not test performance since last time, I don't remember I had special vm options passed in at that time. ------------- PR Comment: https://git.openjdk.org/jdk/pull/18605#issuecomment-2195132669