This is the mail archive of the
newlib@sourceware.org
mailing list for the newlib project.
Re: [PATCH 2/5] add base fenv.h and default sys/fenv.h
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