RFR (S): 8037344: Use the "next" field to iterate over fine remembered instead of using the hash table

Thomas Schatzl thomas.schatzl at oracle.com
Thu Mar 20 09:14:19 UTC 2014


Hi all,

On Fri, 2014-03-14 at 17:41 +0100, Thomas Schatzl wrote: 
> Hi all,
> 
>   could I have reviews for this small change that cleans up the
> HeapRegionRemSet-iterator?
> 

  Jesper had a look at the changes, with two comments:

heapRegionRemSet.hpp: add a newline between line 433 and 434

 433   SparsePRTIter _sparse_iter;
 434  public:

heapRegionRemSet.cpp: 

1122   HeapWord* r_bot =
1123     _fine_cur_prt->hr()->bottom();

remove the line break here.

Fixed in the new webrev here:
http://cr.openjdk.java.net/~tschatzl/8037344/webrev.1/

Thanks,
  Thomas





More information about the hotspot-gc-dev mailing list