Request for review: 6979444

David Holmes David.Holmes at
Mon Aug 30 09:27:02 PDT 2010

Ivan Krylov said the following on 08/30/10 23:26:
>   Bug Link:
> Webrev:

The actual link in your html went to the wrong webrev :)

but the real webrev seems ok to me.


> The following change effects only nonproduct builds. A new boolean flag 
> is introduced: -XX:+PrintFlagsWithComments
> It works pretty much like -XX:+PrintFlagsInitial (or -Xprintflags) but 
> adds a doc string to each flag that is present in every flag definition 
> in the corresponding *global*.hpp file.
> Minor corrections were made to Flag::print_on method to do proper 
> formatting of all 3 flags (more space for flag name and type, printing 
> of double default values).
> Thanks,
> Ivan

More information about the hotspot-runtime-dev mailing list