cvs commit: src/sys/i386/include _types.h

Bruce Evans brde at optusnet.com.au
Thu Mar 6 03:15:41 UTC 2008


On Wed, 5 Mar 2008, Colin Percival wrote:

> Bruce Evans wrote:
>>   Modified files:
>>     sys/i386/include     _types.h
>>   Log:
>>   Change float_t and double_t to long double on i386.
>
> Doesn't this have a rather severe performance impact on any code which
> uses double_t?

No.  As mentioned in the commit message, this has no performance effect
except in cases where it avoids compiler bugs.  In fact, I need it
partly to improve performance in cases where other methods of avoiding
the compiler bugs (including fixing the compiler) have a severe
performance effect.  Using a correctly defined double_t makes it
possible for non-broken generated code to operate entirely on registers,
while using plain double requires either broken generated code or
lots of conversions via memory.  E.g.:

 	double x, y, z;
 	z = x + y;

 	Typical wrong generated i386 code for this:
 	fadd	%st(1),%st

 	Non-broken code for this (get it by fixing the compiler or messing
 	up the source code using something like STRICT_ASSIGN()):
 	fadd	%st(1),%st
 	fstpl	mem		# or fstps/flds to convert to float
 	fldl	mem

Using double_t == long double:

 	double_t x, y, z;
 	z = x + y;

 	Typical generated i386 code for this:
 	fadd	%st(1),%st

x and y would probably start out as doubles and be loaded into registers
using fldl.  This gives a long double whether you want it to or not
(all FP registers are long double).  Then the severe performance impact
is from converting the long double back to "double z" on assignment
as is required for C compilers.  OTOH, if you use long double for
memory variables then you get a severe performance impact and some
space loss for the load instruction, since loading long doubles is
much slower than loading doubles (about 4 times slower on Athlons).

Bruce


More information about the cvs-src mailing list