RFR: 8331558: AArch64: optimize integer remainder [v2]
Jin Guojie
duke at openjdk.org
Thu May 30 07:12:31 UTC 2024
> On some Arm processors, a separate multiply/subtract is actually faster than the combined instruction.
>
> (1) The following test has passed, which shows performance improvement.
>
> make test TEST="micro:java.lang.IntegerDivMod"
> make test TEST="micro:java.lang.LongDivMod"
>
> * IntegerDivMod.testDivideRemainderUnsigned baseline(ns/ops) 2223 with this pacth(ns/ops) 1885 improvement(%) 17.93%
>
> * IntegerDivMod.testRemainderUnsigned baseline(ns/ops) 2225 with this pacth(ns/ops) 1885 improvement(%) 18.03%
>
> * LongDivMod.testDivideRemainderUnsigned baseline(ns/ops) 2231 with this pacth(ns/ops) 1894 improvement(%) 17.79%
>
> * LongDivMod.testRemainderUnsigned baseline(ns/ops) 2232 with this pacth(ns/ops) 1891 improvement(%) 18.03%
>
> (2) jtreg test has passed
>
> make run-test TEST=tier1
Jin Guojie 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 two additional commits since the last revision:
- Merge branch 'openjdk:master' into dev0530
- 8331558: AArch64: optimize integer remainder
On some Arm processors, a separate multiply/subtract is actually faster than the combined instruction.
(1) The following test has passed, which shows performance improvement.
make test TEST="micro:java.lang.IntegerDivMod"
make test TEST="micro:java.lang.LongDivMod"
* IntegerDivMod.testDivideRemainderUnsigned
baseline(ns/ops) 2223
with this pacth(ns/ops) 1885
improvement(%) 17.93%
* IntegerDivMod.testRemainderUnsigned
baseline(ns/ops) 2225
with this pacth(ns/ops) 1885
improvement(%) 18.03%
* LongDivMod.testDivideRemainderUnsigned
baseline(ns/ops) 2231
with this pacth(ns/ops) 1894
improvement(%) 17.79%
* LongDivMod.testRemainderUnsigned
baseline(ns/ops) 2232
with this pacth(ns/ops) 1891
improvement(%) 18.03%
(2) jtreg test has passed
make run-test TEST=tier1
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/19471/files
- new: https://git.openjdk.org/jdk/pull/19471/files/12af7ac0..21af82e4
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=19471&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=19471&range=00-01
Stats: 1143 lines in 39 files changed: 673 ins; 235 del; 235 mod
Patch: https://git.openjdk.org/jdk/pull/19471.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/19471/head:pull/19471
PR: https://git.openjdk.org/jdk/pull/19471
More information about the hotspot-dev
mailing list