RFR: 8334755: Asymptotically faster implementation of square root algorithm [v17]

fabioromano1 duke at openjdk.org
Thu Jun 27 20:08:42 UTC 2024


> I have implemented the Zimmermann's square root algorithm, available in works [here](https://inria.hal.science/inria-00072854/en/) and [here](https://www.researchgate.net/publication/220532560_A_proof_of_GMP_square_root).
> 
> The algorithm is proved to be asymptotically faster than the Newton's Method, even for small numbers. To get an idea of how much the Newton's Method is slow,  consult my article [here](https://arxiv.org/abs/2406.07751), in which I compare Newton's Method with a version of classical square root algorithm that I implemented. After implementing Zimmermann's algorithm, it turns out that it is faster than my algorithm even for small numbers.

fabioromano1 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 47 additional commits since the last revision:

 - Merge branch 'openjdk:master' into patchSqrt
 - Added "throw" to throw the ArithmeticException created
 - Correct BigDecimal.sqrt()
 - Simplified computing square root of BigDecimal using BigInteger.sqrt()
 - Removed unnecessary variable
 - Optimized to compute the remainder only if needed
 - Optimized multiplication
 - Code optimization
 - Merge branch 'openjdk:master' into patchSqrt
 - Removed useless instruction
 - ... and 37 more: https://git.openjdk.org/jdk/compare/b0e0dbc8...d3ca0d4f

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/19710/files
  - new: https://git.openjdk.org/jdk/pull/19710/files/549c0969..d3ca0d4f

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=19710&range=16
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=19710&range=15-16

  Stats: 7267 lines in 207 files changed: 4535 ins; 1905 del; 827 mod
  Patch: https://git.openjdk.org/jdk/pull/19710.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/19710/head:pull/19710

PR: https://git.openjdk.org/jdk/pull/19710


More information about the core-libs-dev mailing list