RFR: 8274069: Clean up g1ParScanThreadState a bit [v3]
Kim Barrett
kbarrett at openjdk.java.net
Tue Sep 21 15:25:38 UTC 2021
On Tue, 21 Sep 2021 14:45:15 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:
>> Hi all,
>>
>> can I have reviews for this change that does some minor refactoring for issues found in JDK-8271880/PR#5037:
>>
>> - remove declaration without definition
>> - move inlined code in the .hpp file into .inline.hpp file
>> - factor out some code
>>
>> Testing: gha, local compilation, local gc/g1
>>
>> Thanks,
>> Thomas
>
> Thomas Schatzl has updated the pull request incrementally with one additional commit since the last revision:
>
> ayang review, fixed comment
Does this change allow this inclusion of an .inline.hpp to be moved out of the .hpp?
33 #include "gc/g1/heapRegionRemSet.inline.hpp"
src/hotspot/share/gc/g1/g1ParScanThreadState.hpp line 132:
> 130:
> 131: // Apply the post barrier to the given reference field. Enqueues the card of p
> 132: // if the barrier (same region, not from survivor) does not filter out not find
I don't understand this comment; seems like there's a grammar or wording problem.
-------------
Changes requested by kbarrett (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/5607
More information about the hotspot-gc-dev
mailing list