* kbarrett review

This commit is contained in:
Thomas Schatzl 2026-01-26 10:42:34 +01:00
parent bcc72bdb73
commit d46255518f

View File

@ -411,12 +411,13 @@ const MemRegion* G1CMRootMemRegions::claim_next() {
return nullptr; return nullptr;
} }
if (_claimed_root_regions.load_relaxed() >= num_root_regions()) { uint local_num_root_regions = num_root_regions();
if (_claimed_root_regions.load_relaxed() >= local_num_root_regions) {
return nullptr; return nullptr;
} }
size_t claimed_index = _claimed_root_regions.fetch_then_add(1u); size_t claimed_index = _claimed_root_regions.fetch_then_add(1u);
if (claimed_index < num_root_regions()) { if (claimed_index < local_num_root_regions) {
return &_root_regions[claimed_index]; return &_root_regions[claimed_index];
} }
return nullptr; return nullptr;
@ -427,7 +428,8 @@ uint G1CMRootMemRegions::num_root_regions() const {
} }
bool G1CMRootMemRegions::contains(const MemRegion mr) const { bool G1CMRootMemRegions::contains(const MemRegion mr) const {
for (uint i = 0; i < num_root_regions(); i++) { uint local_num_root_regions = num_root_regions();
for (uint i = 0; i < local_num_root_regions; i++) {
if (_root_regions[i].equals(mr)) { if (_root_regions[i].equals(mr)) {
return true; return true;
} }