RFR (XS): 8073654: Marking statistics should use size_t
Thomas Schatzl
thomas.schatzl at oracle.com
Mon Mar 9 09:28:30 UTC 2015
Hi Sangheon,
On Thu, 2015-03-05 at 17:23 -0800, Sangheon Kim wrote:
> 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
>
In concurrentMark.cpp, there is now an extra space at the end of line
292. I do not need a new webrev for fixing that.
Looks good.
I can sponsor it if you still need a sponsor.
Thanks,
Thomas
More information about the hotspot-gc-dev
mailing list