hg: valhalla/valhalla: 8222711: [lworld] Initial skeletal implementation of inline class instance construction via <init>

Harold Seigel harold.seigel at oracle.com
Mon Apr 29 15:56:43 UTC 2019


Thanks Srikanth!

Harold

On 4/29/2019 11:54 AM, Srikanth wrote:
>
>
> On 29/04/19 7:22 PM, Harold Seigel wrote:
>> Hi,
>>
>> Here's a slightly updated version of John's patch to support static 
>> <init> factory methods for inline types.
>>
>> Open Webrev: http://cr.openjdk.java.net/~hseigel/lworld_8222787/webrev/
>>
>> Please note the following:
>>
>> 1. The patch hard-wires the Javac noStaticInitValueFactory option to
>>    always be 'on'.
>
> I think you are trying to say, unless invoked with 
> -XDnoStaticInitValueFactory, javac will produce the new style static 
> <init> factories.
>
> I am OK with this change on the javac side subjecting to a clean run 
> of the langtools/tools/javac/valhalla tests at the time of the push 
> post review - They were clean when we both ran them individually.
>
> Srikanth
>



More information about the valhalla-dev mailing list