review request (M) - 6578152: fill_region_with_object

John Coomes John.Coomes at sun.com
Tue Nov 25 22:49:26 UTC 2008


John Coomes (John.Coomes at sun.com) wrote:
> 6578152: fill_region_with_object has usability and safety issues
> 
> http://webrev.invokedynamic.info/jcoomes/6578152-fill/
> 
> Touches 25 files, but many are trivial changes.  Thanks for any
> comments.

Ping :-).  Still looking for reviews.

I updated the webrev after two minor changes:  (a) the new versions of
insert_deadspace() in psMarkSweepDecorator.cpp and space.cpp now mark
the object live as the old versions did, and (b) fixed a typo in
fill_with_array in collectedHeap.cpp (no semantic change).  If anyone
looked at the previous version and would like more details on these,
let me know.

-John

P.S.:  I'll be on vacation until Tue, Dec 2.




More information about the hotspot-gc-dev mailing list