RFR (S): 8066102: Clean up HeapRegionRemSet files

Mikael Gerdin mikael.gerdin at oracle.com
Fri Nov 28 12:10:00 UTC 2014


Hi Thomas,

On 2014-11-28 09:46, Thomas Schatzl wrote:
> Hi all,
>
>    can I have reviews for this small cleanup? I went over heap region
> remembered set files recently and found dead code, useless code,
> tightened the public interface and tried to document the methods a
> little. I also moved FromCardCache accessor methods from
> OtherRegionTable to HeapRegionRemSet to do away with that needless
> indirection. Imo it fits better that way too.
>
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8066102
>
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/8066102/webrev/

In heapRegionRemSet.hpp:
+  // One bits in the bitmaps indicate that the given region or card is 
live.
typo: "One bit in one of the bitmaps indicate that the.."

Otherwise the change looks good to me.

/Mikael

>
> Testing:
> jprt
>
> Thanks,
>    Thomas
>



More information about the hotspot-gc-dev mailing list