fluencelabs/musl
1
0
Fork 0
mirror of https://github.com/fluencelabs/musl synced 2025-07-01 23:52:02 +00:00
Code Issues Projects Releases Wiki Activity

math: use double_t for temporaries to avoid stores on i386

Browse Source 
When FLT_EVAL_METHOD!=0 (only i386 with x87 fp) the excess
precision of an expression must be removed in an assignment.
(gcc needs -fexcess-precision=standard or -std=c99 for this)

This is done by extra load/store instructions which adds code
bloat when lot of temporaries are used and it makes the result
less precise in many cases.
Using double_t and float_t avoids these issues on i386 and
it makes no difference on other archs.

For now only a few functions are modified where the excess
precision is clearly beneficial (mostly polynomial evaluations
with temporaries).

object size differences on i386, gcc-4.8:
             old   new
__cosdf.o    123    95
__cos.o      199   169
__sindf.o    131    95
__sin.o      225   203
__tandf.o    207   151
__tan.o      605   499
erff.o      1470  1416
erf.o       1703  1649
j0f.o       1779  1745
j0.o        2308  2274
j1f.o       1602  1568
j1.o        2286  2252
tgamma.o    1431  1424
math/*.o   64164 63635
This commit is contained in:
Szabolcs Nagy
2013-05-15 23:08:52 +00:00
parent 2c184264ea
commit e216951f50
21 changed files with 31 additions and 28 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/math/j0f.c
View File

@ -201,7 +201,7 @@ static const float pS2[5] = {
static float pzerof(float x)
{
const float *p,*q;
float z,r,s;
float_t z,r,s;
uint32_t ix;
GET_FLOAT_WORD(ix, x);
@ -297,7 +297,7 @@ static const float qS2[6] = {
static float qzerof(float x)
{
const float *p,*q;
float s,r,z;
float_t s,r,z;
uint32_t ix;
GET_FLOAT_WORD(ix, x);