RFR: 8229372: Static inline fields should allow circularity.

Frederic Parain frederic.parain at oracle.com
Wed Aug 14 19:16:22 UTC 2019


Thank you for the review.

Fred


> On Aug 14, 2019, at 08:59, David Simms <david.simms at oracle.com> wrote:
> 
> All in all it looks good...
> 
> I did need to re-read the code back (a second time) from src/hotspot/share/interpreter/interpreterRuntime.cpp:400, the point was the CHECK macros as per the intro comment should throw and the fatal not be reached. fatal() message says this.
> 
> Cheer, thanks for the fix Frederic !
> 
> On 2019-08-13 21:42, Frederic Parain wrote:
>> Greeting,
>> 
>> Here’s a changeset to relax the circularity rules
>> for static fields.
>> 
>> The handling of erroneous situations is not great,
>> but relaxing rules could allow explorations at
>> the language level. Error handling would have to
>> be fixed later (it requires to store the error on
>> first initialization failure, in order to throw exactly
>> the same error on further attempts to initialize the field).
>> 
>> CR: https://bugs.openjdk.java.net/browse/JDK-8229372
>> 
>> Webrev: http://cr.openjdk.java.net/~fparain/statics2/webrev.00/index.html
>> 
>> Thank you,
>> 
>> Fred



More information about the valhalla-dev mailing list