diff options
author | jcoomes <none@none> | 2008-12-11 12:05:08 -0800 |
---|---|---|
committer | jcoomes <none@none> | 2008-12-11 12:05:08 -0800 |
commit | 2abc760b0cd13b9569a73f37b24e7239f966b052 (patch) | |
tree | 6967cbeb0c1ab6fce00b9d49b966333a713db2c9 /src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp | |
parent | c3ada4f8fc8280640edd6e76af92303b6f776b64 (diff) | |
download | jdk8u_hotspot-2abc760b0cd13b9569a73f37b24e7239f966b052.tar.gz |
6578152: fill_region_with_object has usability and safety issues
Reviewed-by: apetrusenko, ysr
Diffstat (limited to 'src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp')
-rw-r--r-- | src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp b/src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp index f0cad8ed2..47bff6cca 100644 --- a/src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp +++ b/src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp @@ -2954,7 +2954,7 @@ public: // The object has been either evacuated or is dead. Fill it with a // dummy object. MemRegion mr((HeapWord*)obj, obj->size()); - SharedHeap::fill_region_with_object(mr); + CollectedHeap::fill_with_object(mr); _cm->clearRangeBothMaps(mr); } } @@ -3225,7 +3225,7 @@ void G1CollectedHeap::par_allocate_remaining_space(HeapRegion* r) { // Otherwise, try to claim it. block = r->par_allocate(free_words); } while (block == NULL); - SharedHeap::fill_region_with_object(MemRegion(block, free_words)); + fill_with_object(block, free_words); } #define use_local_bitmaps 1 @@ -3619,9 +3619,8 @@ public: guarantee(alloc_buffer(purpose)->contains(obj + word_sz - 1), "should contain whole object"); alloc_buffer(purpose)->undo_allocation(obj, word_sz); - } - else { - SharedHeap::fill_region_with_object(MemRegion(obj, word_sz)); + } else { + CollectedHeap::fill_with_object(obj, word_sz); add_to_undo_waste(word_sz); } } |