RFR: 8311588: C2: RepeatCompilation compiler directive does not choose stress seed randomly
Tobias Hartmann
thartmann at openjdk.org
Mon Jul 10 05:19:01 UTC 2023
On Thu, 6 Jul 2023 15:11:05 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:
> The compiler directive `RepeatCompilation` can be used to repeat compilations of specific methods, for example for `Test::test()`:
>
> -XX:CompileCommand=RepeatCompilation,Test::test,1000
>
> When using this compiler directive in combination with `StressIGVN/CCP/GCM/LCM`, C2 only sets the stress seed once to a random value. Afterward, it keeps the seed for all repeated compilations. This should be changed to always select a random seed with each repetition (i.e. get the same behavior as the flag `RepeatCompilation`).
>
> Thanks,
> Christian
Looks good and trivial to me.
-------------
Marked as reviewed by thartmann (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/14786#pullrequestreview-1521230316
More information about the hotspot-compiler-dev
mailing list