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

Roberto Castaneda Lozano roberto.castaneda.lozano at oracle.com
Fri Sep 25 08:02:44 UTC 2020


"Merge master" (c2c31c3) addresses a trivial conflict caused by the 
integration of JDK-8253586 (https://github.com/openjdk/jdk/pull/334).

On 2020-09-25 09:54, Roberto Castaneda Lozano wrote:
>> Add `StressIGVN` option to let C2 randomize IGVN worklist order. When enabled, the worklist is shuffled before each
>> main run of the IGVN loop. Also add `GenerateStressSeed` and `StressSeed=N` options to randomly generate or specify the
>> seed. In either case, the seed is logged if `LogCompilation` is enabled. The new options are declared as
>> production+diagnostic for consistency with the existing `StressLCM` and `StressGCM` options.
> 
> Roberto Castaneda Lozano has updated the pull request with a new target base due to a merge or a rebase. The pull
> request now contains 14 commits:
> 
>   - Merge master
>   - Add missing @requires annotation to test case
>   - Empty commit to trigger jcheck after updating GitHub user name
>   - Generate random seed if 'StressSeed' is unset
>   - Use generic swap() for shuffling
>   - Add basic sanity test for stress IGVN options
>   - Fix typo
>   - Move shuffle() to PhaseIterGVN
>   - Define 'StressSeed' option as 'uint' rather than 'uintx'
>   - Apply minor rearrangements to simplify the patch
>   - ... and 4 more: https://git.openjdk.java.net/jdk/compare/37b70282...c2c31c3e
> 
> -------------
> 
> Changes: https://git.openjdk.java.net/jdk/pull/242/files
>   Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=242&range=07
>    Stats: 229 lines in 10 files changed: 224 ins; 0 del; 5 mod
>    Patch: https://git.openjdk.java.net/jdk/pull/242.diff
>    Fetch: git fetch https://git.openjdk.java.net/jdk pull/242/head:pull/242
> 
> PR: https://git.openjdk.java.net/jdk/pull/242
> 


More information about the hotspot-compiler-dev mailing list