Request for reviews (M): 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
john.r.rose at oracle.com
Fri Jun 25 18:49:15 PDT 2010
That's good. Thanks for simplifying the code that way.
This line has been obsolete for a long time; you can delete it:
+ // If no Def-Use info existed for 'nn' it will after call.
On Jun 25, 2010, at 6:24 PM, Vladimir Kozlov wrote:
> Fixed 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
> Currently C2 does not check that users of a subsumed node
> are placed on IGVN worklist. Because of that some expressions
> are not collapsed as expected.
> I suggest to use replace_node() method instead of subsume_node().
> Make subsume_node() private and use it only in transform_old().
> Passed CTW, JPRT.
More information about the hotspot-compiler-dev