On Fri, 3 Nov 2023 09:56:43 GMT, Albert Mingkun Yang <ay...@openjdk.org> wrote:
>> Thomas Schatzl has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Add documentation about why and how we handle pinned regions in the >> young/old generation. > > src/hotspot/share/gc/g1/g1GCPhaseTimes.cpp line 482: > >> 480: } >> 481: >> 482: double G1GCPhaseTimes::print_post_evacuate_collection_set(bool >> evacuation_retained) const { > > Why the renaming here? Probably forgot to undo the rename. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/16342#discussion_r1381501087