[PATCH 2/5] add base fenv.h and default sys/fenv.h

Kazunori Asayama asayama@sm.sony.co.jp
Wed Feb 14 17:31:00 GMT 2007


Patrick Mansfield wrote:
> On Wed, Feb 14, 2007 at 12:26:09PM +0900, Kazunori Asayama wrote:
>> Patrick Mansfield <patmans@us.ibm.com> wrote:
>>>> BTW, I don't know how the fenv functions should behave in
>>>> multi-threading environment, i.e., should the fenv functions affect
>>>> only the current thread ? If so, I suppose that fenv status should be
>>>> managed by using _reent structure and that reentrant versions of fenv
>>>> functions should be also declared in fenv.h...
>>> No, since we don't have threading. Right? 
>> Right, but the basic fenv infrastructure, including fenv.h, will be
>> shared between all target platforms. Don't we need to take care about
>> other platforms ?
> 
> I think that should be added if or when required, even so I don't think
> the reentrant structures/functions are needed on most platforms since the
> state is in the hardware.

OK, I agreed. I have no strong opinion about this issue,
if the current implementation is acceptable for the trunk.

-- 
(ASAYAMA Kazunori
  (asayama@sm.sony.co.jp))
t



More information about the Newlib mailing list