Re: RFR: 8275821: Optimize random number generators developed in JDK-8248862 using Math.unsignedMultiplyHigh() [v6]

2021-12-03 Thread Jim Laskey 
On Fri, 3 Dec 2021 02:14:51 GMT, Vamsi Parasa  wrote:

>> This change optimizes random number generators using 
>> Math.unsignedMultiplyHigh()
>
> Vamsi Parasa has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   minor changes to the benchmark

LGTM

-

Marked as reviewed by jlaskey (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/6206

Re: RFR: 8275821: Optimize random number generators developed in JDK-8248862 using Math.unsignedMultiplyHigh() [v6]

2021-12-02 Thread Paul Sandoz 
On Fri, 3 Dec 2021 02:11:24 GMT, Vamsi Parasa  wrote:

>> This change optimizes random number generators using 
>> Math.unsignedMultiplyHigh()
>
> Vamsi Parasa has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   minor changes to the benchmark

Marked as reviewed by psandoz (Reviewer).

-

PR: https://git.openjdk.java.net/jdk/pull/6206

Re: RFR: 8275821: Optimize random number generators developed in JDK-8248862 using Math.unsignedMultiplyHigh() [v6]

2021-12-02 Thread Vamsi Parasa 
> This change optimizes random number generators using 
> Math.unsignedMultiplyHigh()

Vamsi Parasa has updated the pull request incrementally with one additional 
commit since the last revision:

  minor changes to the benchmark

-

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/6206/files
  - new: https://git.openjdk.java.net/jdk/pull/6206/files/e9c71477..cf68fe00

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk=6206=05
 - incr: https://webrevs.openjdk.java.net/?repo=jdk=6206=04-05

  Stats: 7 lines in 1 file changed: 1 ins; 3 del; 3 mod
  Patch: https://git.openjdk.java.net/jdk/pull/6206.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/6206/head:pull/6206

PR: https://git.openjdk.java.net/jdk/pull/6206