Integrated: 8277878: Fix compiler tests after JDK-8275908
Volker Simonis
simonis at openjdk.java.net
Mon Nov 29 12:33:10 UTC 2021
On Sat, 27 Nov 2021 09:53:26 GMT, Volker Simonis <simonis at openjdk.org> wrote:
> This is a quick fix for the two compiler tests introduced by JDK-8275908. The test explicitly added SerialGC in the parameter list which leads to a garbage collector conflict, if the tests are run with some other Garbage collector.
>
> It was suggested to fix this by adding a `@requires vm.gc.Serial` tag but this is not necessary because the tests are actually GC-agnostic so I've removed the `-XX:+UseSerialGC` parameter from the test command line instead.
>
> After the fix it was necessary to refine the check for whether the test JVM has JVMCI support built-in. Before the fix, I used `WhiteBox.getWhiteBox().isJVMCISupportedByGC()` which worked fine if only running with SerialGC. Now that we can run with GCs which don't support JVMCI we have to use the more specific `(WB.getBooleanVMFlag("EnableJVMCI") != null)`.
>
> Please let me know if you want me to push this instantly after it has been reviewed or if you first want to re-run your internal Tier3 tests before pushing.
This pull request has now been integrated.
Changeset: 614c6e61
Author: Volker Simonis <simonis at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/614c6e61fa3a9f094a311b12e780491c611657e6
Stats: 4 lines in 2 files changed: 1 ins; 0 del; 3 mod
8277878: Fix compiler tests after JDK-8275908
Reviewed-by: thartmann, chagedorn
-------------
PR: https://git.openjdk.java.net/jdk/pull/6581
More information about the hotspot-compiler-dev
mailing list