On Wed, 6 Jan 2016, Michael Meissner wrote:
> 5) I put the exception handling code under #ifndef _SOFT_FLOAT
> control so
> that the 32-bit PowerPC machines that don't have hardware floating
> point will not use this code (the exception handling uses hardware
> instructions to cause a particular fault).
I think this needs to be __NO_FPRS__ instead of _SOFT_FLOAT (since soft-fp
is also built for e500).
Other than that my only comments on this version are some formatting /
coding style comments:
> Index: libgcc/config/rs6000/sfp-exceptions.c
> ===================================================================
> --- libgcc/config/rs6000/sfp-exceptions.c
> (.../svn+ssh://meiss...@gcc.gnu.org/svn/gcc/trunk/libgcc) (revision 0)
> +++ libgcc/config/rs6000/sfp-exceptions.c (.../libgcc) (revision
> 232096)
> @@ -0,0 +1,74 @@
> +/*
> + * Copyright (C) 2016 Free Software Foundation, Inc.
> + *
Normally we don't use the '*' at the start of each comment line.
> +TFtype
> +__floatsikf_hw (SItype_ppc a)
> +{
> + return (TFtype)a;
Casts should have a space, "(TFtype) a". Many more cases in this file.
> +/* __unordkf2 returns 1 if Nan or 0 otherwise. */
NaN.
> + __low = (double) (__value - (__float128)__high);
> \
Again, space in cast.
> + /* now renormalized move the high/low into canonical IBM long \
> + double form. */ \
Comments start with a capital letter.
> + /* Handle the special cases of NAN and inifinity. */
> \
"infinity".
> + RESULT = ((__float128)__high) + ((__float128)__low); \
Spaces in casts.
--
Joseph S. Myers
jos...@codesourcery.com
