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

Srikanth srikanth.adayapalam at oracle.com
Mon Apr 29 15:54:56 UTC 2019



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