RFR (12) JDK-8218025: disable pop_frame and force_early_return caps for Graal

Igor Ignatyev igor.ignatyev at oracle.com
Thu Jan 31 01:34:37 UTC 2019

Hi Alex,

UseJVMCICompiler is declared only if INCLUDE_JVMCI is defined, so your current patch will break builds where it's undefined. the rest (esp. problem list ;) ) looks good to me.

adding hotspot-compiler alias.

-- Igor

> On Jan 30, 2019, at 5:27 PM, Alex Menkov <alexey.menkov at oracle.com> wrote:
> Hi all,
> Please review a fix for tck-red bug:
> https://bugs.openjdk.java.net/browse/JDK-8218025
> webrev:
> http://cr.openjdk.java.net/~amenkov/tck_red_disable_caps/webrev/
> ForceEarlyReturn and PopFrame JCK tests intermittently fail with Graal.
> Real fix for the issue is too risky for jdk12, so we have to disable pop_frame and force_early_return capabilities running with Graal (the capabilities are optional).
> Currently Graal is the only compiler, so the fix checks if JVMCI compiler is enabled.
> JCK test passes with disabled capabilities.
> A number of hotspot tests do not check if the capabilities are enabled (as hotspot is expected to support all capabilities) and fail with Graal - they are problem-listed until the real problem (JDK-8195635) is resolved.
> --alex

More information about the hotspot-compiler-dev mailing list