Integrated: 8254805: compiler/debug/TestStressCM.java is still failing

Roberto Castañeda Lozano rcastanedalo at openjdk.java.net
Tue Oct 20 06:12:14 UTC 2020


On Fri, 16 Oct 2020 06:57:33 GMT, Roberto Castañeda Lozano <rcastanedalo at openjdk.org> wrote:

> Use the code motion trace produced by `TraceOptoPipelining` (excluding traces of stubs) to assert that two compilations
> with the same seed cause `StressLCM` and `StressGCM` to take the same randomized decisions. Previously, the entire
> output produced by `PrintOptoStatistics` was used instead, which has shown to be too fragile. Also, disable inlining in
> both `TestStressCM.java` and the similar `TestStressIGVN.java` to prevent flaky behavior, and run both tests for ten
> different seeds to improve coverage.

This pull request has now been integrated.

Changeset: 98ec4a67
Author:    Roberto Castañeda Lozano <roberto.castaneda.lozano at oracle.com>
Committer: Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/98ec4a67
Stats:     34 lines in 2 files changed: 21 ins; 3 del; 10 mod

8254805: compiler/debug/TestStressCM.java is still failing

Use the code motion trace produced by TraceOptoPipelining (excluding traces of
stubs) to assert that two compilations with the same seed cause StressLCM and
StressGCM to take the same randomized decisions. Previously, the entire output
produced by PrintOptoStatistics was used instead, which has shown to be too
fragile. Also, disable inlining in both TestStressCM.java and the similar
TestStressIGVN.java to prevent flaky behavior, and run both tests for ten
different seeds to improve coverage.

Reviewed-by: kvn, thartmann

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

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


More information about the hotspot-compiler-dev mailing list