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