RFR(S): 8187822: C2 conditonal move optimization might create broken graph

Vladimir Kozlov vladimir.kozlov at oracle.com
Fri Sep 29 18:11:13 UTC 2017


On 9/29/17 1:26 AM, Roland Westrelin wrote:
> 
>> My only concern now if there is a case when data node has control edge
>> pointing to branch of diamond and depends on If's condition (NULL
>> check). Can this happen?
> 
> These:
> 
>    // Check for ops pinned in an arm of the diamond.
>    // Can't remove the control flow in this case
>    if (lp->outcnt() > 1) return NULL;
>    if (rp->outcnt() > 1) return NULL;
> 
> prevent it, right?

Yes. Thanks to look on it. Changes are good.

Vladimir

> 
> Roland.
> 


More information about the hotspot-compiler-dev mailing list