RFR(XS): 8209686: cleanup arguments to PhaseIdealLoop() constructor
Vladimir Kozlov
vladimir.kozlov at oracle.com
Mon Aug 20 20:42:00 UTC 2018
Good to me too.
Thanks,
Vladimir
On 8/20/18 2:00 AM, Tobias Hartmann wrote:
> 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