[Bug inline-asm/39059] [4.3/4.4 regression] ICE with fixed-point type in inline-asm

[Bug inline-asm/39059] [4.3/4.4 regression] ICE with fixed-point type in inline-asm

[reichelt at gcc dot gnu dot org]

-- 

reichelt at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.3.4


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

[Bug inline-asm/39059] New: [4.3/4.4 regression] ICE with fixed-point type in inline-asm

[reichelt at gcc dot gnu dot org]

The following (IMHO valid) testcase triggers an ICE since GCC 4.3.0
(when fixed-point types were introduced):

============================================================
void foo()
{
  asm("" : : "r"(0r));
}
============================================================

bug.c: In function 'foo':
bug.c:4: internal compiler error: in init_move_cost, at reginfo.c:270
Please submit a full bug report, [etc.]

Technically, this is a regression since the code did not crash the
compiler before (because it was rejected earlier).


-- 
           Summary: [4.3/4.4 regression] ICE with fixed-point type in
                    inline-asm
           Product: gcc
           Version: 4.4.0
            Status: UNCONFIRMED
          Keywords: ice-on-valid-code, monitored
          Severity: normal
          Priority: P3
         Component: inline-asm
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: reichelt at gcc dot gnu dot org


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

[Bug inline-asm/39059] [4.3/4.4 regression] ICE with fixed-point type in inline-asm

[rguenth at gcc dot gnu dot org]

-- 

rguenth at gcc dot gnu dot org changed:

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


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

[Bug inline-asm/39059] [4.3/4.4 regression] ICE with fixed-point type in inline-asm

[jakub at gcc dot gnu dot org]

------- Comment #1 from jakub at gcc dot gnu dot org  2009-02-03 10:06 -------
Similarly
__typeof (0r)
foo (void)
{
  return 0r;
}
ICEs on x86-64 and so does
_Fract foo (void)
{
  return 0r;
}
(the last one just as error-recovery ICE).  I'd say for
!targetm.fixed_point_supported_p () we should remove the loop hole that
fixed-point constants introduce fixed point types without any error, similarly
for C++ we should reject FIXED_CSTs, and to improve error-recovery, we
shouldn't used fixed point types after we've diagnosed we don't support them.


-- 

jakub at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |jakub at gcc dot gnu dot org
                   |dot org                     |
             Status|UNCONFIRMED                 |ASSIGNED
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2009-02-03 10:06:51
               date|                            |


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

[Bug inline-asm/39059] [4.3/4.4 regression] ICE with fixed-point type in inline-asm

[jakub at gcc dot gnu dot org]

------- Comment #2 from jakub at gcc dot gnu dot org  2009-02-03 17:26 -------
Subject: Bug 39059

Author: jakub
Date: Tue Feb  3 17:26:28 2009
New Revision: 143900

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=143900
Log:
        PR inline-asm/39059
        * c-parser.c (c_parser_postfix_expression): If fixed point is not
        supported, don't accept FIXED_CSTs.
        * c-decl.c (finish_declspecs): Error if fixed point is not supported
        and _Sat is used without _Fract/_Accum.  Set specs->type to
        integer_type_node for cts_fract/cts_accum if fixed point is not
        supported.

        * parser.c (cp_parser_primary_expression): Reject FIXED_CSTs.

        * gcc.dg/nofixed-point-2.c: New test.
        * g++.dg/ext/fixed1.C: Adjust expected diagnostics.
        * g++.dg/ext/fixed2.C: Likewise.
        * g++.dg/other/error25.C: Likewise.
        * g++.dg/lookup/crash7.C: Likewise.
        * g++.dg/cpp0x/decltype-38655.C: Likewise.

Added:
    trunk/gcc/testsuite/gcc.dg/nofixed-point-2.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/c-decl.c
    trunk/gcc/c-parser.c
    trunk/gcc/cp/ChangeLog
    trunk/gcc/cp/parser.c
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/g++.dg/cpp0x/decltype-38655.C
    trunk/gcc/testsuite/g++.dg/ext/fixed1.C
    trunk/gcc/testsuite/g++.dg/ext/fixed2.C
    trunk/gcc/testsuite/g++.dg/lookup/crash7.C
    trunk/gcc/testsuite/g++.dg/other/error25.C


-- 


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

[Bug inline-asm/39059] [4.3 regression] ICE with fixed-point type in inline-asm

[jakub at gcc dot gnu dot org]

------- Comment #3 from jakub at gcc dot gnu dot org  2009-02-03 19:30 -------
Fixed on the trunk.


-- 

jakub at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[4.3/4.4 regression] ICE    |[4.3 regression] ICE with
                   |with fixed-point type in    |fixed-point type in inline-
                   |inline-asm                  |asm


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

[Bug inline-asm/39059] [4.3 regression] ICE with fixed-point type in inline-asm

[rguenth at gcc dot gnu dot org]

------- Comment #4 from rguenth at gcc dot gnu dot org  2009-08-04 12:29 -------
GCC 4.3.4 is being released, adjusting target milestone.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.3.4                       |4.3.5


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

[Bug inline-asm/39059] [4.3 regression] ICE with fixed-point type in inline-asm

[rguenth at gcc dot gnu dot org]

------- Comment #5 from rguenth at gcc dot gnu dot org  2010-05-22 18:13 -------
GCC 4.3.5 is being released, adjusting target milestone.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.3.5                       |4.3.6


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