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