RFR: 8229797: [JVMCI] Clean up no longer used JVMCI::dependencies_invalid value.
coleen.phillimore at oracle.com
coleen.phillimore at oracle.com
Wed Aug 28 21:40:30 UTC 2019
With my limited knowledge of Graal, this looks good. It's consistent
with the other JITs now.
Coleen
On 8/28/19 12:51 PM, dean.long at oracle.com wrote:
> Looks good.
>
> dl
>
> On 8/27/19 7:25 PM, Xiaohong Gong (Arm Technology China) wrote:
>> 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 hotspot-dev
mailing list