Request for reviews (S): 6671250: In Parse::do_if() old Cmp node 'c' should be replaced with new one after BoolNode transformation.

Vladimir Kozlov Vladimir.Kozlov at Sun.COM
Tue Mar 4 17:52:51 PST 2008


http://webrev.invokedynamic.info/kvn/6671250/index.html

Fixed 6671250: In Parse::do_if() old Cmp node 'c' should be replaced
                with new one after BoolNode transformation.

Problem:
In Parse::do_if() 'c' (CmpNode) node may be changed during BoolNode
transformation so 'c' may became dead but the node is referenced
later in the code.

Solution:
'c' should be replaced with new node after BoolNode transformation.

Reviewed by:
Fix verified (y/n): y

Other testing:
PRT




More information about the hotspot-compiler-dev mailing list