RFR: 8290525: Move HeapRegion::_compaction_top to G1FullCollector [v2]
    Thomas Schatzl 
    tschatzl at openjdk.org
       
    Thu Jul 21 08:02:05 UTC 2022
    
    
  
> Hi all,
> 
>   can I have reviews for this change that moves `HeapRegion::_compaction_top` into the `G1FullCollector` class; my reasoning is that these values are transient temporary values only ever useful for the full collection algorithm, basically results of the application of the algorithm, that are otherwise unused. This makes `HeapRegion` a bit easier to understand/read.
> 
> Testing: tier1-3
> 
> Thanks,
>   Thomas
Thomas Schatzl has updated the pull request incrementally with one additional commit since the last revision:
  kbarrett review
-------------
Changes:
  - all: https://git.openjdk.org/jdk/pull/9560/files
  - new: https://git.openjdk.org/jdk/pull/9560/files/16d264f6..57d69c14
Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=9560&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=9560&range=00-01
  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/9560.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/9560/head:pull/9560
PR: https://git.openjdk.org/jdk/pull/9560
    
    
More information about the serviceability-dev
mailing list