[14] RFR(S): 8231412: C2: InitializeNode::detect_init_independence() bails out on simple IR shapes

Christian Hagedorn christian.hagedorn at oracle.com
Tue Oct 22 10:42:22 UTC 2019


Thank you Tobias and Roland for your reviews!

On 22.10.19 11:11, Roland Westrelin wrote:
> Is this still required?
> 
> 3553       uint first_i = n->find_edge(m);
> 3554       if (i != first_i)  continue;  // process duplicate edge just once
> 
> If m was already processed, it's armless to push it on the work list
> again because it's a Unique_Node_List so the check above is redundant?

You're right. This check is not necessary anymore with a 
Unique_Node_List. I updated the webrev again:
http://cr.openjdk.java.net/~chagedorn/8231412/webrev.01/

Best regards,
Christian


More information about the hotspot-compiler-dev mailing list