review for 6796786: invalid FP identity transform - (a - b) -> b (original) (raw)

Tom Rodriguez tom.rodriguez at oracle.com
Tue May 3 12:12:52 PDT 2011

Thanks Christian and Igor.

tom

On May 3, 2011, at 1:11 AM, Christian Thalinger wrote:

On May 2, 2011, at 11:25 PM, Tom Rodriguez wrote:

http://cr.openjdk.java.net/~never/6796786

6796786: invalid FP identity transform - (a - b) -> b - a Reviewed-by: C2 is transforming the idiom - (a - b) into b - a which produces the wrong sign if a == b. The fix is to eliminate that optimization. Tested with test from report. Looks good. -- Christian

More information about the hotspot-compiler-dev mailing list