On Thu, Nov 19, 2009 at 9:29 AM, John M McIntosh < johnmci@smalltalkconsulting.com> wrote:
define FLT_EPSILON as something like 1e-20 and rewrite the check as
CHECK(fabs(f - ff) < FLT_EPSILON)?
In math epsilon is a small value tending towards zero. The intent here
is to check that you get back something close to ff without assuming that the floating-point arithmetic involved in many is exact enough to return ff.
Ok, the implication then for testing is that
ffiPushSingleFloat ffiReturnFloatValue
modify the floating point value slightly, if that is acceptable then you need to epsilon test.
No. Look at the body of many:
static float many(float f1, float f2, float f3, float f4, float f5, float f6, float f7, float f8, float f9, float f10, float f11, float f12, float f13, float f14, float f15) { DPRINTF(("%f %f %f %f %f %f %f %f %f %f %f %f %f %f %f\n", (double)f1, (double)f2, (double)f3, (double)f4, (double)f5, (double)f6, (double)f7, (double)f8, (double)f9, (double)f10, (double)f11, (double)f12, (double)f13, (double)f14, (double)f15)); return ((f1/f2+f3/f4+f5/f6+f7/f8+f9/f10+f11/f12+f13/f14) * f15); }
There's lots of scope for floating-point rounding errors here, so the epsilon test is required. There's no implication that any float marshalling code alters the values involved. That would be a huge bug.
also GCC seems to define #define FLT_EPSILON 1.19209290e-07F across all flavours of CPUs and float word lengths. perhaps via "float.h"
--
John M. McIntosh johnmci@smalltalkconsulting.com Twitter: squeaker68882 Corporate Smalltalk Consulting Ltd. http://www.smalltalkconsulting.com ===========================================================================