This is the mail archive of the cygwin-patches mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Add fenv.h and support.


On Fri, Sep 10, 2010 at 09:53:28PM +0100, Dave Korn wrote:
>
>
>    Hi folks,
>
>  This patch adds fenv.h and the related support routines in the Cygwin DLL.
>It's an entirely unencumbered implementation that I wrote from scratch using
>only the public docs for reference.  We've been missing this for a while, what
>with PR323 and all, and if we add it in we'll be able to switch on the new
>decimal-floating-point features in the compiler.  (Amongst I'm sure many other
>uses).
>
>winsup/cygwin/ChangeLog:
>
>	* Makefile.in (DLL_OFILES): Add new fenv.o module.
>	(fenv_CFLAGS): New flags definition for fenv.o compile.
>	* autoload.cc (std_dll_init): Use fenv.h functions instead of direct
>	manipulation of x87 FPU registers.
>	* crt0.c (mainCRTStartup): Likewise.
>	* cygwin.din (feclearexcept, fegetexceptflag, feraiseexcept,
>	fesetexceptflag, fetestexcept, fegetround, fesetround, fegetenv,
>	feholdexcept, fesetenv, feupdateenv, fegetprec, fesetprec,
>	feenableexcept, fedisableexcept, fegetexcept, _feinitialise,
>	_fe_dfl_env, _fe_nomask_env): Export new functions and data items.
>	* fenv.cc: New file.
>	* posix.sgml: Update status of newly-implemented APIs.
>	* include/fenv.h: Likewise related header.
>	* include/cygwin/version.h (CYGWIN_VERSION_API_MINOR): Bump.
>
>  Testing: well, I'm running the GCC testsuite against it to verify it builds
>functioning decimal floating point code, and I've manually tested some of the
>simple functionality like setting the exceptions on and off.  That's all so
>far, but I think it's close enough (and given that it's new functionality) to
>check in and fix any bugs that crop up on HEAD.  (I'd like to also see if I
>can run some of the LSB or Posix verification testsuites against it, but I
>don't know what's involved in that yet; if anyone has any experience with any
>of that stuff, I'd appreciate being dropped a note off-list with a few pointers.)

Looks nice to me with one HUGE caveat:  Please maintain the pseudo-sorted
order in cygwin.din.  Sorry to have to impose this burden on you.

Other than that, please check in and thanks for the patch.  It was obviously
a lot of work.

cgf


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]