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