RFR: 8080947: Add uint as a valid VM flag type

Bengt Rutisson bengt.rutisson at oracle.com
Tue Jun 2 12:14:42 UTC 2015


Hi David,

On 2015-05-28 13:28, David Lindholm wrote:
> Hi,
>
> Please review this patch that adds uint and int as valid VM flag 
> types. This patch adds the possibility to specify VM flags with types 
> int and uint, it does not change the type of any flags.
>
>
> Webrev: http://cr.openjdk.java.net/~david/JDK-8080947/webrev.hotspot.00/
> Webrev: http://cr.openjdk.java.net/~david/JDK-8080947/webrev.jdk.00/
> Bug: https://bugs.openjdk.java.net/browse/JDK-8080947

Overall this looks good to me, but shouldn't Flag::get_uint() in 
globals.cpp/hpp return an uint rather than an int?

  113 int Flag::get_uint() const {

  287   int get_uint() const;

Thanks,
Bengt



>
>
> Testing: Passed JPRT
>
>
> Thanks,
> David



More information about the hotspot-dev mailing list