From aleksey.shipilev at oracle.com  Tue Aug  2 10:12:11 2016
From: aleksey.shipilev at oracle.com (aleksey.shipilev at oracle.com)
Date: Tue, 02 Aug 2016 10:12:11 +0000
Subject: hg: code-tools/jol: 7901019: jol estimates do not take
	AllocationStyle into the account
Message-ID: <201608021012.u72ACBIp020243@aojmv0008.oracle.com>

Changeset: d93b948ed60e
Author:    shade
Date:      2016-08-02 13:11 +0300
URL:       http://hg.openjdk.java.net/code-tools/jol/rev/d93b948ed60e

7901019: jol estimates do not take AllocationStyle into the account
Summary: Rewrite HotSpotLayouter to match actual VM allocator, amend with gaps strategies, more tests.
Contributed-by: Timur Gibadullin <timur.gbd at gmail.com>

! jol-cli/src/main/java/org/openjdk/jol/operations/HeapDump.java
! jol-cli/src/main/java/org/openjdk/jol/operations/ObjectEstimates.java
! jol-cli/src/main/java/org/openjdk/jol/operations/ObjectIdealPacking.java
! jol-cli/src/main/java/org/openjdk/jol/operations/StringCompress.java
! jol-core/src/main/java/org/openjdk/jol/info/ClassData.java
! jol-core/src/main/java/org/openjdk/jol/info/ClassLayout.java
! jol-core/src/main/java/org/openjdk/jol/info/FieldData.java
+ jol-core/src/main/java/org/openjdk/jol/layouters/FieldAllocationType.java
! jol-core/src/main/java/org/openjdk/jol/layouters/HotSpotLayouter.java
+ jol-core/src/main/java/org/openjdk/jol/vm/ContendedSupport.java
+ jol-core/src/test/java/org/openjdk/jol/layouters/HotspotLayouterRealTest.java
! jol-core/src/test/java/org/openjdk/jol/layouters/LayouterInvariantsTest.java
! jol-core/src/test/java/org/openjdk/jol/util/ClassGenerator.java


From aleksey.shipilev at oracle.com  Tue Aug  2 10:14:04 2016
From: aleksey.shipilev at oracle.com (Aleksey Shipilev)
Date: Tue, 2 Aug 2016 13:14:04 +0300
Subject: [PATCH] 7901019: jol estimates do not take AllocationStyle into
	the account
In-Reply-To: <5786B6C3.8030907@gmail.com>
References: <CAPqVdRigoif9SHD28ijLjFQaEfmVTjzap=MJZVTO8p8-SEH2_A@mail.gmail.com>
	<CAPqVdRhURRkt+XcZ_V0QJUDW0mG-xVVVO9sCoC6qDrgsinUf_Q@mail.gmail.com>
	<56A2029B.5080909@oracle.com>
	<AEA1B9E1-BFF1-4B9B-9A71-A4E9431D373E@gmail.com>
	<56A653F0.4060708@oracle.com> <56A6B258.1080807@gmail.com>
	<570F8C38.9040801@oracle.com>
	<CAPqVdRiVdC-nROs1JpB0dKOrRaFpphSO0Jys9cU5y+TKfjD3Qg@mail.gmail.com>
	<57549E0B.2020307@gmail.com> <5755A335.4080200@oracle.com>
	<5776F967.3020705@gmail.com> <577846DF.8050808@gmail.com>
	<57790D4B.7090406@gmail.com> <57791213.8030508@gmail.com>
	<577986E5.7000905@gmail.com> <5786B6C3.8030907@gmail.com>
Message-ID: <3257da90-2bf6-0a8e-0653-d06b43bd5bb3@oracle.com>

On 07/14/2016 12:46 AM, ????? ?????????? wrote:
> Just realized, that the SuperGaps strategy is extension of the
> HierarchyGaps strategy, in the previous patch the takeSuperGaps could
> not handle gaps, that could be handled by takeHierarchyGaps option.
> Fixed this in the new patch.

Thank you, seems to be working fine. I pushed and resolved:
  https://bugs.openjdk.java.net/browse/CODETOOLS-7901019

-Aleksey