8269914: Factor out heap printing for G1 young and full gc

[tschatzl]

Hi all,

can I have reviews for this change that factors out before/after heap related printing into a single scoped object to be used for both young and full gc code?

There is some minor change in order of unrelated options (COMMIT/UNCOMMIT region state changes vs. other heap printing), during full gc but since this is very low level debugging I do not see a problem to do that.

Based on #4539 .

Thanks,
Thomas

---

Progress

• Change must not contain extraneous whitespace
• Commit message must refer to an issue
• Change must be properly reviewed

Integration blocker

 ⚠️ Dependency #4539 must be integrated first

Issue

• JDK-8269914: Factor out heap printing for G1 young and full gc

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.java.net/jdk pull/4689/head:pull/4689
$ git checkout pull/4689

Update a local copy of the PR:
$ git checkout pull/4689
$ git pull https://git.openjdk.java.net/jdk pull/4689/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 4689

View PR using the GUI difftool:
$ git pr show -t 4689

Using diff file

Download this PR as a diff file:
https://git.openjdk.java.net/jdk/pull/4689.diff

[bridgekeeper]

👋 Welcome back tschatzl! A progress list of the required criteria for merging this PR into pr/4539 will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

[openjdk]

@tschatzl The following label will be automatically applied to this pull request:

• hotspot-gc

When this pull request is ready to be reviewed, an "RFR" email will be sent to the corresponding mailing list. If you would like to change these labels, use the /label pull request command.

[mlbridge]

Webrevs

• 00: Full (0f3881b1)