RFR (XS): 8073654: Marking statistics should use size_t

Jesper Wilhelmsson jesper.wilhelmsson at oracle.com
Fri Mar 6 10:26:38 UTC 2015


Looks good.

Just curious, what motivated the first change, moving the variables in 
concurrentMark.cpp?

Also, your webrevs would be easier to review if you used the latest version of 
webrev found here:
http://hg.openjdk.java.net/code-tools/webrev

Thanks,
/Jesper


Sangheon Kim skrev den 6/3/15 02:23:
> Hi all,
>
> Can I have reviews for this simple fix for changing variables for statistics
> from 'int' to 'size_t'.
> This is to avoid overflows on large heaps.
>
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8073654
> Webrev:
> http://cr.openjdk.java.net/~sangheki/8073654/webrev.00/
> Testing:
> JPRT
>
> Thanks,
> Sangheon



More information about the hotspot-gc-dev mailing list