RFR: 8300769: Remove G1CollectionSet::_inc_bytes_used_before [v3]

Thomas Schatzl tschatzl at openjdk.org
Tue Jan 24 17:38:30 UTC 2023


On Tue, 24 Jan 2023 06:16:03 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:

>> Thomas Schatzl has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains three commits:
>> 
>>  - Merge branch 'master' into pull/12113
>>  - ayang review, remove redundant _bytes_used_before
>>  - initial version
>
> src/hotspot/share/gc/g1/g1YoungGCPostEvacuateTasks.cpp line 468:
> 
>> 466:   void report(G1CollectedHeap* g1h, G1EvacInfo* evacuation_info) {
>> 467:     evacuation_info->set_regions_freed(_regions_freed);
>> 468:     evacuation_info->set_collection_set_used_before(_after_used_bytes + _before_used_bytes);
> 
> The ordering, `before + after`, is more natural, IMO.

Fixed

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

PR: https://git.openjdk.org/jdk/pull/12113


More information about the hotspot-gc-dev mailing list