--- a/src/hotspot/share/gc/g1/g1HotCardCache.cpp Mon Nov 25 12:30:24 2019 +0100
+++ b/src/hotspot/share/gc/g1/g1HotCardCache.cpp Mon Nov 25 12:31:39 2019 +0100
@@ -68,7 +68,7 @@
return card_ptr;
}
// Otherwise, the card is hot.
- size_t index = Atomic::add(1u, &_hot_cache_idx) - 1;
+ size_t index = Atomic::add(&_hot_cache_idx, 1u) - 1;
size_t masked_index = index & (_hot_cache_size - 1);
CardValue* current_ptr = _hot_cache[masked_index];
@@ -91,8 +91,8 @@
assert(!use_cache(), "cache should be disabled");
while (_hot_cache_par_claimed_idx < _hot_cache_size) {
- size_t end_idx = Atomic::add(_hot_cache_par_chunk_size,
- &_hot_cache_par_claimed_idx);
+ size_t end_idx = Atomic::add(&_hot_cache_par_claimed_idx,
+ _hot_cache_par_chunk_size);
size_t start_idx = end_idx - _hot_cache_par_chunk_size;
// The current worker has successfully claimed the chunk [start_idx..end_idx)
end_idx = MIN2(end_idx, _hot_cache_size);