[15] RFR(M): 8240227: Loop predicates should be copied to unswitched loops

Christian Hagedorn christian.hagedorn at oracle.com
Mon Mar 16 10:59:56 UTC 2020


Thank you Tobias for your review!

Best regards,
Christian

On 16.03.20 11:56, Tobias Hartmann wrote:
> Hi Christian,
> 
> On 13.03.20 13:10, Christian Hagedorn wrote:
>>> - loopPredicate.cpp:415/422 Why are two versions of PhaseIdealLoop::clone_loop_predicates required?
>>
>> There was originally a method PhaseIterGVN::clone_loop_predicates which did not have any uses. I
>> removed it in the current webrev. But in addition, we can also remove one of those remaining
>> clone_loop_predicates methods which is not required anymore and make all involved methods
>> non-static. I removed all related dead code.
> 
> Nice!
> 
>> I created new updated webrevs with the cleaned up code:
>> Full: http://cr.openjdk.java.net/~chagedorn/8240227/webrev.01/
>> Inc: http://cr.openjdk.java.net/~chagedorn/8240227/webrev.00-01/
> 
> Looks good!
> 
> Best regards,
> Tobias
> 


More information about the hotspot-compiler-dev mailing list