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