RFR: 8326227: Rounding error that may distort computeNextGaussian results [v3]

Chris Hennick duke at openjdk.org
Wed Feb 21 02:18:08 UTC 2024


> This provides a slightly more accurate bounding limit for `computeNextExponentialSoftCapped` when the computed bound is greater than `(1.0p53 - 1.0) * DoubleZigguratTables.exponentialX0`. This could cause the `while (computeNextExponentialSoftCapped(rng, limit) < limit)` check in `computeNextGaussian` on line 1402 to always be true, making the `nextGaussian` runtime unbounded in the worst case.

Chris Hennick has updated the pull request incrementally with one additional commit since the last revision:

  Bug fix: add-exports was for wrong package

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/17703/files
  - new: https://git.openjdk.org/jdk/pull/17703/files/e68d03eb..b8be051c

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=17703&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=17703&range=01-02

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/17703.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/17703/head:pull/17703

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


More information about the core-libs-dev mailing list