RFR: 8250961: Move Universe::update_heap_info_at_gc to CollectedHeap [v2]

[Stefan Karlsson]

On Tue, 8 Sep 2020 23:55:53 GMT, Aditya Mandaleeka <adityam at openjdk.org> wrote:

>> This is a small refactoring that moves `update_heap_info_at_gc()` and related members to `CollectedHeap`. There should
>> be no behavioral change.
>> jdk/hotspot tier1 passed
>> 
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8250961
>
> Aditya Mandaleeka has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Rename and rearrange things a bit per feedback.

Thanks. Just two small nits below. You still need a second reviewer before this can be integrated.

src/hotspot/share/gc/shared/collectedHeap.hpp line 248:

> 246:   // Historic gc information
> 247:   size_t free_at_last_gc()    { return _capacity_at_last_gc - _used_at_last_gc; }
> 248:   size_t used_at_last_gc()    { return _used_at_last_gc; }

There are some unnecessary spaces between () and {. Could you also make these functions const?

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

Marked as reviewed by stefank (Reviewer).

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