RFR(XS): 8211451: ~2.5% regression on compression benchmark starting with 12-b11

Vladimir Kozlov vladimir.kozlov at oracle.com
Tue Nov 6 16:41:55 UTC 2018


I think code in insert_loop_limit_check() is not matching previous code:

now:

+  _igvn.replace_input_of(iff, 1, bol);

before:

-    _igvn.replace_input_of(opq, 1, bol);
-    // Update ctrl.
-    set_ctrl(opq, check_iff->in(0));
-    set_ctrl(check_iff->in(1), check_iff->in(0));


Other changes looks good.

Thanks,
Vladimir

On 11/6/18 5:02 AM, Roland Westrelin wrote:
> 
> Hi Vladimir,
> 
> Here is a new webrev that implements the extra runtime check you
> suggested:
> 
> http://cr.openjdk.java.net/~roland/8211451/webrev.01/
> 
> Roland.
> 


More information about the hotspot-compiler-dev mailing list