Request for reviews (M): 6677629: PhaseIterGVN::subsume_node() should call hash_delete() and add_users_to_worklist()
vladimir.kozlov at oracle.com
Mon Jun 28 14:16:28 PDT 2010
Thank you, Tom
Tom Rodriguez wrote:
> That's a much needed cleanup. It looks good.
> 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