RFR 8134995(M): [REDO] GC: implement ranges (optionally constraints) for those flags that have them missing

sangheon.kim sangheon.kim at oracle.com
Fri Sep 25 17:42:16 UTC 2015


Hi Kim, Jon and Zoltan,

Here's next version of webrev.
This includes:
- changed from '4' to 'wordSize' at g1_globals.hpp. (Kim and Jon? )
- removed test code which is already commented out at 
TestG1HeapRegionSize.java. (Zoltan)

webrev:
cr.openjdk.java.net/~sangheki/8134995/webrev.01
cr.openjdk.java.net/~sangheki/8134995/webrev.01_to_00

Thanks,
Sangheon


On 09/24/2015 10:39 AM, Kim Barrett wrote:
> On Sep 23, 2015, at 7:57 PM, sangheon.kim <sangheon.kim at oracle.com> wrote:
>>> I see now.  jintSize would be better than 4.  But I think even better would be pointer size,
>>> though I haven’t found an existing constant for that.  Maybe sizeof(address)?
>> We have 'const int wordSize = sizeof(char*)'.
>>
>>> The only
>>> disadvantage to that is the max value is then smaller on 64bit platforms than on 32bit
>>> platforms.  I doubt the difference is interesting in practice though.
>> I agree that there's no big difference in practice.
>> I just wanted to avoid to have smaller upper limit for 64bit platforms.
>> But if you prefer to use 'wordSize' instead of '4', I'm fine. It would give better readability.
> Yes, please.
>



More information about the hotspot-dev mailing list