On Fri, Jul 29, 2016 at 05:37:26PM +0000, Joseph Myers wrote:
> Ping.  This patch 
> <https://gcc.gnu.org/ml/gcc-patches/2016-07/msg01516.html> 
> (non-C-front-end parts) is pending review.

I'm starting to look at the patches.  It will take some time to come up to
speed, particularly with the other stuff I'm doing.

In your targhooks.{c,h} patches, you have code like:

+       case 32:
+#ifdef HAVE_SFmode
+         cand = SFmode;
+#endif
+         break;

Do you need to check whether the format for SFmode is IEEE?

        case 32:
#ifdef HAVE_SFmode
          if (REAL_MODE_FORMAT (SFmode) == ieee_single_format)
            cand = SFmode;
#endif
          break;

Or is the check at the end whether ieee_bits > (for extended tests) or == (for
normal tests) sufficient to say that SFmode, DFmode, really is IEEE?

-- 
Michael Meissner, IBM
IBM, M/S 2506R, 550 King Street, Littleton, MA 01460-6245, USA
email: meiss...@linux.vnet.ibm.com, phone: +1 (978) 899-4797

Reply via email to