RFR: 8229797: [JVMCI] Clean up no longer used JVMCI::dependencies_invalid value.
Xiaohong Gong (Arm Technology China)
Xiaohong.Gong at arm.com
Wed Aug 28 01:54:05 UTC 2019
Hi,
Please help to review this jvmci patch:
Webrew: http://cr.openjdk.java.net/~pli/rfr/8229797/webrev.00/
JBS: https://bugs.openjdk.java.net/browse/JDK-8229797
This patch fix issue: https://github.com/oracle/graal/issues/1587.
The loading of new classes can cause dependencies to become false, which requires the dependent
nmethods to be discarded and deoptimized. So if validating dependencies fails, it should make the
result to be JVMCI::dependencies_failed, which makes jvmci throw the BailoutException.
The invalid dependencies happen at the time of installation without any intervening modification of
the system dictionary. So as the system dictionary modification optimization has been removed, the
compiler can not know whether the failed dependencies are triggered by class reloading or not. It's
better to use dependencies_failed to mark the result.
Thanks,
Xiaohong Gong
More information about the graal-dev
mailing list