On Thu, 17 Aug 2023 15:51:47 GMT, Aleksey Shipilev <sh...@openjdk.org> wrote:
>> Current test expects `G1HeapRegionSize` flag to be printed out from jmap.
>> But that one is not printed when Shenandoah is enabled. I opted for the
>> minimally intrusive fix: move the additional Shenandoah printout to the
>> Shenandoah-specific block.
>>
>> Additional testing:
>> - [x] Affected test now passes with Shenandoah, continues to pass with G1,
>> Serial, Parallel
>> - [x] `sun/tools/jhsdb` passes with {Serial, Parallel, G1, Shenandoah}
>> - [x] `serviceability/sa` passes with {Serial, Parallel, G1, Shenandoah}
>
> Aleksey Shipilev has updated the pull request incrementally with one
> additional commit since the last revision:
>
> More fixes
Marked as reviewed by cjplummer (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/15328#pullrequestreview-1582961269
