[PATCH 2/2] ARM: Improve fenv implementation
Joseph S. Myers
joseph@codesourcery.com
Tue May 6 22:39:00 GMT 2014
On Fri, 25 Apr 2014, Wilco wrote:
> The second patch improves the fenv implementation by using the inline
> functions from fenv_private where possible rather than duplicating
> functionality. The remaining functions are optimized in a similar way to
> avoid unnecessary FPSCR writes (but no longer use __glibc_unlikely like
> in the previous version).
>
> OK for commit?
OK, but I think further rearrangement in feupdateenv will end up being
needed to fix bug 16918 (libc_feupdateenv_vfp returns void and doesn't
test whether exception traps were successfully enabled, so the approach in
your implementation is not well-suited to handling the FE_NOMASK_ENV case
properly, although it matches the buggy handling in the existing code).
--
Joseph S. Myers
joseph@codesourcery.com
More information about the Libc-alpha
mailing list