RFR: 8252219: C2: Randomize IGVN worklist for stress testing [v3]

Vladimir Kozlov kvn at openjdk.java.net
Wed Sep 23 15:28:49 UTC 2020


On Wed, 23 Sep 2020 11:36:12 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> Roberto Castañeda Lozano has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Define 'StressSeed' option as 'uint' rather than 'uintx'
>
> src/hotspot/share/opto/c2_globals.hpp line 55:
> 
>> 53:           "Randomize worklist traversal in IGVN")                           \
>> 54:                                                                             \
>> 55:   product(bool, GenerateStressSeed, false, DIAGNOSTIC,                      \
> 
> Is this flag really required? We could simply generate the seed if StressSeed has not been specified on the command
> line (see `FLAG_IS_DEFAULT` macro).

Agree.

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

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


More information about the hotspot-compiler-dev mailing list