Request for reviews (S): 6686791: Side effect in NumberFormat tests with -server -Xcomp
Vladimir.Kozlov at Sun.COM
Fri Apr 11 16:17:57 PDT 2008
Fixed 6686791: Side effect in NumberFormat tests with -server -Xcomp
New optimization in CmpPNode::sub() (6667580) removed the valid compare
instruction because of false positive answer from detect_dominating_control()
since one of CmpP's inputs (LoadP) has a control edge above the second input
Check control edges of all 'dom' node's inputs instead of checking
only the control edge.
Remove the fixed (20) steps limit, use NodeList to check
visited Region nodes instead.
New code gives 4 times more correct positive answers based on
jbb2005 and jvm98 runs. It doesn't affect compilation time noticeably.
Benchmarks do not show regression.
Fix verified (y/n): y
More information about the hotspot-compiler-dev