review for 6796786: invalid FP identity transform - (a - b) -> b - a
Igor Veresov
igor.veresov at oracle.com
Mon May 2 15:13:43 PDT 2011
Looks good.
igor
On 5/2/11 2: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.
>
More information about the hotspot-compiler-dev
mailing list