RFR: 8258772: Some runtime/cds tests fail with +LogCompilation or +StressX [v2]

Tobias Hartmann thartmann at openjdk.java.net
Thu Jan 7 10:16:10 UTC 2021


On Thu, 7 Jan 2021 10:06:26 GMT, Roberto Castañeda Lozano <rcastanedalo at openjdk.org> wrote:

>> Emit warning about TraceDependencies results in `ClassHierarchyWalker::count_find_witness_calls()` only if `TraceDependencies` is actually enabled. Use standard `warning()` function instead of ad hoc printing. Remove warning about using `Stress(LCM|GCM|IGVN)` without `LogCompilation` in `Compile::Compile()`, and add the information to the description of the `StressSeed` option instead. These changes prevent false test failures when using `LogCompilation` or `Stress(LCM|GCM|IGVN)`.
>
> Roberto Castañeda Lozano has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
> 
>  - Update copyright year
>  - Merge branch 'master' into JDK-8258772
>  - 8258772: Some runtime/cds tests fail with +LogCompilation or +StressX
>    
>    Emit warning about TraceDependencies results in
>    ClassHierarchyWalker::count_find_witness_calls() only if TraceDependencies is
>    actually enabled. Use standard warning() function instead of ad hoc printing.
>    Remove warning about using Stress(LCM|GCM|IGVN) without LogCompilation in
>    Compile::Compile(), and add the information to the description of the StressSeed
>    option instead. These changes prevent false test failures when using
>    LogCompilation or Stress(LCM|GCM|IGVN).

Marked as reviewed by thartmann (Reviewer).

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

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


More information about the hotspot-compiler-dev mailing list