[OpenJDK Rasterizer] RFR: Marlin renderer #3

[Laurent Bourgès]

Jim,

Here is a new webrev to improve FloatMath:
http://cr.openjdk.java.net/~lbourges/marlin/marlin-s3.0/

I tested it with following float values:
floats       = [-2.13422758E9, -1.37992608E8, -134758.4, -131.5, -17.2,
-1.9, -0.9, -1.0E-4, -1.0E-8, -1.0E-23, -100.0, -3.0, -1.0, -0.0, 0.0, 0.0,
1.0, 3.0, 100.0, 131.5, 17.2, 1.9, 0.9, 1.0E-4, 1.0E-8, 1.0E-23,
2.13422758E9, 1.37992608E8, 134758.4, NaN, -Infinity, -5.0E20, 5.0E20,
Infinity]

strictMathCeil   = [-2134227584, -137992608, -134758, -131, -17, -1, 0, 0,
0, 0, -100, -3, -1, 0, 0, 0, 1, 3, 100, 132, 18, 2, 1, 1, 1, 1, 2134227584,
137992608, 134759, 0, -2147483648, -2147483648, 2147483647, 2147483647]

floatMathCeil    = [-2134227584, -137992608, -134758, -131, -17, -1, 0, 0,
0, 0, -100, -3, -1, 0, 0, 0, 1, 3, 100, 132, 18, 2, 1, 1, 1, 1, 2134227584,
137992608, 134759, 0, -2147483648, -2147483648, 2147483647, 2147483647]

The previous implementation was not correct with +/-Infinity and NaN.


Please note I found one bug very similar to my changes:
https://bugs.openjdk.java.net/browse/JDK-8091651

Do you think it is worth to discuss ceil/floor(float) optimizations with
core-libs ?

Laurent
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/graphics-rasterizer-dev/attachments/20150619/4317d7c1/attachment.html>