[Bug target/25042] New: __float128 ICE on x86

[Bug target/25042] New: __float128 ICE on x86

[jsm28 at gcc dot gnu dot org]

Compile the following test (reduced from
gcc.dg/torture/fp-int-convert-float128.c) on i686-pc-linux-gnu with -O -mmmx. 
It gives an ICE

z.c: In function 'main':
z.c:9: internal compiler error: in cgraph_local_info, at cgraph.c:525
Please submit a full bug report,
with preprocessed source if appropriate.
See <URL:http://gcc.gnu.org/bugs.html> for instructions.

I don't know if __float128 is meant to be supported on x86 (provided the user
provides the external library with the relevant functions) or not (it's an
optional part of the x86_64 ABI), but it should not ICE in any case; it should
generate meaningful code or a meaningful error.  (-mmmx is required for the
__float128 type to be available on x86.)

extern void abort (void);
extern void exit (int);

int
main (void)
{
  static volatile int i;
  static volatile __float128 a;
  a = i;
  if (0 != a)
    abort ();
  exit (0);
}


-- 
           Summary: __float128 ICE on x86
           Product: gcc
           Version: 4.2.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: jsm28 at gcc dot gnu dot org
GCC target triplet: i?86-*-*


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25042

[Bug target/25042] [3.4/4.0/4.1/4.2 Regression] __float128 ICE on x86

[pinskia at gcc dot gnu dot org]

------- Comment #1 from pinskia at gcc dot gnu dot org  2005-11-27 01:00 -------
Hmm, 3.3.x did not define __float128 at all.  3.4 and 4.0 did not ICE at -O1
-mmmx but did at -O2 -mmmx (that means it is unit-at-a-time related).  I am
going to mark this as a regression as anything to an ICE should be considered a
regression but I am not going to confirm it as I don't know which way this
should be, rejected or accepted.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|__float128 ICE on x86       |[3.4/4.0/4.1/4.2 Regression]
                   |                            |__float128 ICE on x86
   Target Milestone|---                         |4.0.3


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25042

[Bug target/25042] [3.4/4.0/4.1/4.2 Regression] __float128 ICE on x86

[mmitchel at gcc dot gnu dot org]

------- Comment #2 from mmitchel at gcc dot gnu dot org  2005-12-19 18:15 -------
Crashes are bad: P2 for now.  If it turns out that this is "just" an
ICE-on-invalid, then we might downgrade the priority.


-- 

mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P2


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25042

[Bug target/25042] [3.4/4.0/4.1/4.2 Regression] __float128 ICE on x86

[hubicka at gcc dot gnu dot org]

------- Comment #3 from hubicka at gcc dot gnu dot org  2005-12-19 23:11 -------
testing patch:
Index: config/i386/i386.c
===================================================================
*** config/i386/i386.c  (revision 108753)
--- config/i386/i386.c  (working copy)
*************** ix86_function_regparm (tree type, tree d
*** 2179,2184 ****
--- 2179,2186 ----

        /* Use register calling convention for local functions when possible. 
*/
        if (!TARGET_64BIT && !user_convention && decl
+         /* Libcalls pass IDENTIFIER_NODE as an decl.  */
+         && TREE_CODE (decl) == FUNCTION_DECL
          && flag_unit_at_a_time && !profile_flag)
        {
          struct cgraph_local_info *i = cgraph_local_info (decl);


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25042

[Bug target/25042] [3.4/4.0/4.1/4.2 Regression] __float128 ICE on x86

[steven at gcc dot gnu dot org]

------- Comment #4 from steven at gcc dot gnu dot org  2006-01-10 20:26 -------
Honza, are you going to do something useful with your patch from comment #3?


-- 

steven at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hubicka at gcc dot gnu dot
                   |                            |org
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
           Keywords|                            |patch
   Last reconfirmed|0000-00-00 00:00:00         |2006-01-10 20:26:18
               date|                            |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25042

[Bug target/25042] [3.4/4.0/4.1/4.2 Regression] __float128 ICE on x86

[hubicka at gcc dot gnu dot org]

------- Comment #5 from hubicka at gcc dot gnu dot org  2006-01-11 13:26 -------
Subject: Bug 25042

Author: hubicka
Date: Wed Jan 11 13:26:45 2006
New Revision: 109582

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=109582
Log:
        PR target/25042
        * i386.c (ix86_init_mmx_sse_builtins): Do not delcare float128

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/i386/i386.c


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25042

[Bug target/25042] [3.4/4.0/4.1/4.2 Regression] __float128 ICE on x86

[hubicka at gcc dot gnu dot org]

------- Comment #6 from hubicka at gcc dot gnu dot org  2006-01-11 13:32 -------
Subject: Bug 25042

Author: hubicka
Date: Wed Jan 11 13:32:44 2006
New Revision: 109583

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=109583
Log:

        PR target/25042
        * i386.c (ix86_init_mmx_sse_builtins): Do not delcare float128

Modified:
    branches/gcc-4_1-branch/gcc/ChangeLog
    branches/gcc-4_1-branch/gcc/config/i386/i386.c


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25042

[Bug target/25042] [3.4/4.0 Regression] __float128 ICE on x86

[pinskia at gcc dot gnu dot org]

------- Comment #7 from pinskia at gcc dot gnu dot org  2006-01-11 14:11 -------
Fixed in 4.1.0 and 4.2.0 at least.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to fail|                            |3.4.0 4.0.0
      Known to work|                            |3.3.3 4.1.0 4.2.0
            Summary|[3.4/4.0/4.1/4.2 Regression]|[3.4/4.0 Regression]
                   |__float128 ICE on x86       |__float128 ICE on x86


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25042

[Bug target/25042] [3.4/4.0 Regression] __float128 ICE on x86

[mmitchel at gcc dot gnu dot org]

-- 

mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.0.3                       |4.0.4


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25042

[Bug target/25042] [4.0 Regression] __float128 ICE on x86

[gdr at gcc dot gnu dot org]

------- Comment #8 from gdr at gcc dot gnu dot org  2007-02-03 16:03 -------
Fixed in GCC-4.1.0.


-- 

gdr at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|4.0.4                       |4.1.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25042