On Fri, 3 Dec 2021 00:17:04 GMT, Vamsi Parasa <d...@openjdk.java.net> 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: > > Update the JMH micro to take RNG parameters for elegant implementation Very nice, much neater, just a few more minor comments. test/micro/org/openjdk/bench/java/util/RandomGeneratorNext.java line 47: > 45: public class RandomGeneratorNext { > 46: > 47: public RandomGenerator randomGenerator; Suggestion: RandomGenerator randomGenerator; test/micro/org/openjdk/bench/java/util/RandomGeneratorNext.java line 52: > 50: String randomGeneratorName; > 51: > 52: public static long[] buffer; Suggestion: long[] buffer; test/micro/org/openjdk/bench/java/util/RandomGeneratorNext.java line 69: > 67: > 68: @Benchmark > 69: @Fork(1) Why is `@Fork` need here? test/micro/org/openjdk/bench/java/util/RandomGeneratorNext.java line 70: > 68: @Benchmark > 69: @Fork(1) > 70: public void testFillBufferWithNextLong() { Return `buffer` after the loop completes, just in case the JIT decides it is otherwise dead code ------------- PR: https://git.openjdk.java.net/jdk/pull/6206