RFR(XS): 8209686: cleanup arguments to PhaseIdealLoop() constructor

Tobias Hartmann tobias.hartmann at oracle.com
Mon Aug 20 09:00:06 UTC 2018


Hi Roland,

nice cleanup, looks good to me.

There is a stray white space at the end of loopnode.cpp:2667

Best regards,
Tobias

On 20.08.2018 10:32, Roland Westrelin wrote:
> 
> http://cr.openjdk.java.net/~roland/8209686/webrev.00/
> 
> Instead of using 3 boolean arguments to PhaseIdealLoop(), I propose
> using a single enum argument that designate what level of loop opts to
> execute. This is another small change from the shenandoah repo where we
> have a few extra ways of triggering loop opts.
> 
> Roland.
> 


More information about the hotspot-compiler-dev mailing list