RFR[L]: 8237767 Field layout computation overhaul

[Frederic Parain]

Hi Aleksey,

Thank you for giving this change a try.

I’ve updated and the two RFRs as duplicates, and I’ve changed the flag name.
The new webrev is here (with updates from David’s review):
http://cr.openjdk.java.net/~fparain/jdk_layout/webrev.05/index.

Regards,

Fred


> On Jan 24, 2020, at 03:46, Aleksey Shipilev <shade at redhat.com> wrote:
> 
> Hi Frederic,
> 
> On 1/23/20 4:03 PM, Frederic Parain wrote:
>> CR: https://bugs.openjdk.java.net/browse/JDK-8237767
>> webrev: http://cr.openjdk.java.net/~fparain/jdk_layout/webrev.04/
>> 
>> The CR includes a detailed description of the motivation and the implementation.
> 
> I ran the patched jdk through JOL Samples [1], and it works impressively well. I think these RFRs
> can be closed as duplicates now:
>  https://bugs.openjdk.java.net/browse/JDK-8024912
>  https://bugs.openjdk.java.net/browse/JDK-8024912
> 
> Nit: I think the flag should be UseNew*Field*Layout.
> 
> -- 
> Thanks,
> -Aleksey
> 
> [1]
> https://hg.openjdk.java.net/code-tools/jol/file/f1e978b895a0/jol-samples/src/main/java/org/openjdk/jol/samples
>