RFR (XS): 8027553 - Change the in_cset_fast_test functionality to use the G1BiasedArray abstraction

[Thomas Schatzl]

Hi Mikael,

On Mon, 2014-03-17 at 16:14 +0100, Mikael Gerdin wrote:
> Thomas,
> 
> On Friday 14 March 2014 10.25.31 Thomas Schatzl wrote:
> > Hi all,
> > 
> >   can I have reviews for the following small change? It cleans up the
> > in_cset_fast_test functionality to use the G1BiasedMappedArray
> > abstraction introduced long ago.
> > 
> > This removes lots of clutter in the code caused by lots of explanation
> > that is part of the G1BiasedMappedArray class description, manual
> > management and asserts all handled by that class.
> > 
> > It is intentional to name the class handling the in_cset_fast_test type
> > G1FastCSetBiasedMappedArray, and not e.g. BoolBiasedMappedArray. The
> > plan is to extend its use for "JDK-8027959: Investigate early
> > reclamation of large objects in G1".
> > 
> > Webrev:
> > http://cr.openjdk.java.net/~tschatzl/8027553/webrev/
> 
> The change looks good.

Thanks for the review,
  Thomas