Request for reviews (S): 6914622: Print values of all flags for product VM
Y. Srinivas Ramakrishna
Y.S.Ramakrishna at Sun.COM
Wed Jan 6 14:30:39 PST 2010
Hi Vladimir --
The following are suggestions/questions: (I do not have
strong views on what you choose to do, and am fine with whatever
(1) For PrintFlagsInitial, i would in the documentation string
in globals,hpp, add that it also exits the vm after printing
(if you choose to retain the exiting behaviour, see below).
Anyone know why we might want to exit the VM after printing the
flags? (i.e. the history of -Xprintflags which you have mimiced here.)
Is there a particular use case this kind of print-and-exit functionality
is targeted at?
(2) I would personally be inclined to delete Xprintflags since it's
not a product flag and can therefore, it seems to me, be deleted
without approval. Why have duplicate functionality/interfaces
even if the duplication is only in debug mode.
Otherwise looks good.
On 01/06/10 14:13, Vladimir Kozlov wrote:
> Fixed 6914622: Print values of all flags for product VM
> Change the flag -XX:+PrintFlagsFinal to product.
> Add new product flag -XX:+PrintFlagsInitial to print all VM flags before
> arguments processing, it is duplicate of debug flag -Xprintflags.
> Move all related print flags together.
> Partially contributed by: Gary Benson
> Note, the changes are different from suggested by Gary Benson
> since I can't change -Xprintflags without approval, so I added
> new option instead.
> Reviewed by:
> Fix verified (y/n): y
> Other testing:
More information about the hotspot-compiler-dev