RFR: 8150720: Cleanup code around PrintOptoStatistics

Claes Redestad claes.redestad at oracle.com
Mon Feb 29 13:16:59 UTC 2016


On 2016-02-27 03:15, Vladimir Kozlov wrote:
> Clearing nearby code is good. You missed {} in ifnode.cpp (no need to 
> resend webrev)
>
> +  if (this == dom)
>      return NULL;

Fixed (and updated in-place[1]).

>
> Yes, changes in node.cpp are good.

Vladimir I. also OK'd this offline, so I'll go ahead and push this version

Thanks for reviewing!

/Claes

[1] http://cr.openjdk.java.net/~redestad/8150720/webrev.01/


More information about the hotspot-compiler-dev mailing list