RFR(XS) 8024917: Remove public pointer to PhaseLive in PhaseChaitin

Vladimir Kozlov vladimir.kozlov at oracle.com
Tue Sep 17 10:02:46 PDT 2013


In general it is fine but I don't like your change to dump() methods.
dump() methods are used during debugging (I used them). Adding 
parameters to them complicates that. Before I could call dump() or 
dump(b) to get info in debugger. Now I have to pass additional argument 
and remember what its local name. Please, find solution to not change 
dump() methods.
And don't remove dump_bb().

thanks,
Vladimir

On 9/17/13 1:58 AM, Niclas Adlertz wrote:
> Hi all,
>
> Removing a public pointer to PhaseLive which can be accessed at any time
> from PhaseChaitin, instead passing a reference of the PhaseLive instance
> to the methods that use it.
>
> A review would be much appreciated.
>
> WEBREV: http://cr.openjdk.java.net/~adlertz/JDK-8024917/webrev00
> JBS: https://bugs.openjdk.java.net/browse/JDK-8024917
>
> Kind Regards,
> Niclas Adlertz


More information about the hotspot-compiler-dev mailing list