RFR: 8260042: G1 Post-cleanup liveness printing occurs too early

Fri Jan 22 09:03:53 UTC 2021

On Wed, 20 Jan 2021 16:00:18 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

> Hi all,
> 
>   can I have reviews for this small change that fixes position of the Post-cleanup liveness printing, causing wrong gc efficiencies to be printed?
> 
> I.e. due to some older changes, the calculation of gc efficiences got moved below the printing of the Post-cleanup liveness which should be about these values. This change corrects that.
> 
> Note that there is a sister issue about not printing the gc efficiencies in the "Post-Marking" phase. This is not scope of this change.
> 
> Testing: manual testing that values are correct, hs-tier1+2

Looks good!

-------------

Marked as reviewed by sjohanss (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/2168