CRR (M): 7127697: G1: remove dead code after recent concurrent mark changes

Tony Printezis tony.printezis at oracle.com
Tue Mar 27 09:29:59 UTC 2012


Hi all,

I'd like a couple of code reviews for this set of changes:

http://cr.openjdk.java.net/~tonyp/7127697/webrev.0/webrev.0.G1CMCleanup/

which removes lots code from conc mark that was made unreachable from a 
recent set of changes. The changes are mostly code removal with only a 
very small number of code actually modified.

This is an additional proposed change:

http://cr.openjdk.java.net/~tonyp/7127697/webrev.0/webrev.1.G1CMCleanupExtra/

to remove a method from the bitmap class which is not used any more (it 
was only used by the code in G1 conc mark that I'm removing). I'd like 
some feedback on whether I should go ahead and remove it or leave it in 
in case it's used in the future.

Tony




More information about the hotspot-gc-dev mailing list