Skip to content

Commit 79a2301

Browse files
committed
8322859: Parallel: Move transform_stack_chunk
Reviewed-by: stefank, tschatzl
1 parent 37e7698 commit 79a2301

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

src/hotspot/share/gc/parallel/psPromotionManager.inline.hpp

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -249,10 +249,6 @@ inline oop PSPromotionManager::copy_unmarked_to_survivor_space(oop o,
249249
// Copy obj
250250
Copy::aligned_disjoint_words(cast_from_oop<HeapWord*>(o), cast_from_oop<HeapWord*>(new_obj), new_obj_size);
251251

252-
// Parallel GC claims with a release - so other threads might access this object
253-
// after claiming and they should see the "completed" object.
254-
ContinuationGCSupport::transform_stack_chunk(new_obj);
255-
256252
// Now we have to CAS in the header.
257253
// Because the forwarding is done with memory_order_relaxed there is no
258254
// ordering with the above copy. Clients that get the forwardee must not
@@ -271,6 +267,8 @@ inline oop PSPromotionManager::copy_unmarked_to_survivor_space(oop o,
271267
assert(young_space()->contains(new_obj), "Attempt to push non-promoted obj");
272268
}
273269

270+
ContinuationGCSupport::transform_stack_chunk(new_obj);
271+
274272
// Do the size comparison first with new_obj_size, which we
275273
// already have. Hopefully, only a few objects are larger than
276274
// _min_array_size_for_chunking, and most of them will be arrays.

0 commit comments

Comments
 (0)