Request for review (xs) - 7112997

Bengt Rutisson bengt.rutisson at oracle.com
Mon Nov 28 07:38:03 UTC 2011


Jon,

I think you removed the only calls to 
ParCompactionManager::should_verify_only() and 
ParCompactionManager::should_reset_only(). Can you remove those methods 
as well? That will propagate to removing the enum values VerifyUpdate 
and ResetObjects in ParCompactionManager::Action.

Other than that it looks good.

Thanks for doing cleanups like this one!
Bengt

On 2011-11-23 20:24, Jon Masamitsu wrote:
> 7112997: Remove obsolete code ResetObjectsClosure and VerifyUpdateClosure
> Summary: Remove obsolete code.
>
> This was debugging code that used  the MarkSweep code to check
> the UseParallelOldGC  compaction.   It has been broken for sometime
> and there are no prospects for fixing it.
>
> http://cr.openjdk.java.net/~jmasa/7112997/webrev.00/
>
> Thanks.
>




More information about the hotspot-gc-dev mailing list