[14] RFR(M): 8233033: Results of execution differ for C2 and C1/Xint

Christian Hagedorn christian.hagedorn at oracle.com
Tue Dec 10 11:43:29 UTC 2019


Hi Roland

You're right, it should be set_ctrl() instead. I changed it and added an 
additional non CFG sanity assertion check:
http://cr.openjdk.java.net/~chagedorn/8233033/webrev.01/

Best regards,
Christian

On 10.12.19 11:46, Roland Westrelin wrote:
> 
> Hi Christian,
> 
>> http://cr.openjdk.java.net/~chagedorn/8233033/webrev.00/
> 
> Is this correct?
> 
> 364         set_idom(stmt, iffast_pred, dom_depth(stmt));
> 366         set_idom(cloned_stmt, ifslow_pred, dom_depth(stmt));
> 
> stmt is a non CFG so it doesn't have an idom but a control?
> 
> Roland.
> 


More information about the hotspot-compiler-dev mailing list