FYI: rework previous fix

Aleksey Shipilev shade at redhat.com
Wed Sep 27 15:25:34 UTC 2017


On 09/27/2017 05:18 PM, Roland Westrelin wrote:
> I intend to push the following change:
> 
> http://cr.openjdk.java.net/~roland/shenandoah/outofloop_wb_correctphi/webrev.00/

This seems to revert our differences around has_only_data_users() against upstream in favor of this
one-liner, right?

 878       } else if (in->Opcode() == Op_CMoveP || in->Opcode() == Op_CMoveN) {

Looks good to me.

Make sure you have descriptive commit message!

Thanks,
-Aleksey



More information about the shenandoah-dev mailing list