Issue399

Title add fenv to libm (was: Re: <EMPTY SUBJECT>)
Priority feature Status chatting
Superseder Nosy List corecode
Assigned To Keywords

Created on 2006-12-07.10:34:16 by corecode, last changed by rumcic.

Messages
msg1714 (view) Author: rumcic Date: 2006-12-08.01:21:00
Done, the patch as before is located at http://www.rumko.net/dragonfly/fenv.diff
msg1713 (view) Author: corecode Date: 2006-12-07.20:56:00
Okay, looks better.  Could you please merge the fields __mxcsr_hi and 
__mxcsr_lo into one uint32_t and remove the compat __[gs]et_mxcsr?  we 
don't have to maintain binary compatibility to freebsd 5.3.

cheers
   simon
msg1712 (view) Author: rumcic Date: 2006-12-07.20:50:01
I got rid of the _types.h files and moved fenv.h to src/lib/libm/arch/i386/

The only changes in the source tree now are 2 new files (fenv.c and fenv.h) 
and the modified Makefile.inc in lib/libm/arch/i386/.

The updated patch is located at http://www.rumko.net/dragonfly/fenv.diff
msg1709 (view) Author: corecode Date: 2006-12-07.10:34:06
okay, this will have to be fixed not to contain types from freebsd.  i am sure that you only need to modify files in lib/libm.

the port to amd64 has already started.  however, it is not finished.  it is crucial that all future work strictly respects this fact and integrates amd64.

for some reason we are now using netbsd's libm, and they don't have special treatment of amd64.  i am pretty sure that is not needed in any case, it is merely an optimization.

to install a cpu-specific header, just add

INCS+= fenv.h

in arch/i386/Makefile.inc

Besides that, the freebsd code contains unnecessary compatibility code which should be removed (like splitting mxcsr).

cheers
  simon
History
Date User Action Args
2006-12-08 01:21:01rumcicsetpriority: feature
messages: + msg1714
2006-12-07 20:56:00corecodesetmessages: + msg1713
2006-12-07 20:50:01rumcicsetstatus: unread -> chatting
messages: + msg1712
2006-12-07 10:34:16corecodecreate