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