RFR(XS): JDK-8186154 : Extra info with +LogTouchedMethods

[coleen.phillimore at oracle.com]

Hi,

I don't think you should add another option to print the count.  You 
should do it unconditionally.  Since it's a diagnostic option, we don't 
have to support parsers of the output.
I was going to type that you should use UL for this output instead, but 
that's a bigger task.  It would be nice if the compiler options used UL 
and didn't have these Print options anymore.  In any case, please don't 
add another Print* option.

thanks,
Coleen

On 8/11/17 3:10 PM, Eric Caspole wrote:
> Hi,
> Please review this very small change to add more info to 
> +LogTouchedMethods including the touched count and whether the method 
> has loops. This extra info has been handy while doing experiments to 
> create the most efficient AOT library for quick startup.
>
> http://cr.openjdk.java.net/~ecaspole/JDK-8186154/webrev/
>
> https://bugs.openjdk.java.net/browse/JDK-8186154
>
> Passed JPRT hotspot tests.
>
> Thanks,
> Eric