[Bug c/43662] New: ICE in insert_save, at caller-save.c:1303

[Bug c/43662] New: ICE in insert_save, at caller-save.c:1303

[marcus at jet dot franken dot de]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 904 bytes --]

during Wine 64bit build I get

gcc -c  -O2      acmstream.i
acmstream.c: In function ‘AVIFILE_OpenCompressor’:
acmstream.c:327:1: internal compiler error: in insert_save, at
caller-save.c:1303
Please submit a full bug report,
with preprocessed source if appropriate.
See <http://bugs.opensuse.org/> for instructions.

gcc version 4.5.0 20100331 (experimental) [trunk revision 157870] (SUSE Linux)


-- 
           Summary: ICE in insert_save, at caller-save.c:1303
           Product: gcc
           Version: 4.5.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: marcus at jet dot franken dot de
 GCC build triplet: x86_64-unknown-linux
  GCC host triplet: x86_64-unknown-linux
GCC target triplet: x86_64-unknown-linux


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

[Bug c/43662] ICE in insert_save, at caller-save.c:1303

[marcus at jet dot franken dot de]

------- Comment #1 from marcus at jet dot franken dot de  2010-04-06 08:35 -------
Created an attachment (id=20322)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20322&action=view)
acmstream.i

gcc -c -O2 acmstream.i

started to reduce...  likely attribute(ms_abi) related I think.


-- 


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

[Bug target/43662] [4.5 Regression] ICE in insert_save, at caller-save.c:1303

[rguenth at gcc dot gnu dot org]

------- Comment #2 from rguenth at gcc dot gnu dot org  2010-04-06 10:03 -------
We use a caller-save slot of VOIDmode for a DImode mem.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
          Component|c                           |target
     Ever Confirmed|0                           |1
           Keywords|                            |ice-on-valid-code
   Last reconfirmed|0000-00-00 00:00:00         |2010-04-06 10:03:20
               date|                            |
            Summary|ICE in insert_save, at      |[4.5 Regression] ICE in
                   |caller-save.c:1303          |insert_save, at caller-
                   |                            |save.c:1303
   Target Milestone|---                         |4.5.1


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

[Bug target/43662] [4.5/4.6 Regression] ICE in insert_save with ms_abi attribute

[pinskia at gcc dot gnu dot org]

------- Comment #3 from pinskia at gcc dot gnu dot org  2010-04-09 02:46 -------
Short testcase:
int __attribute__((ms_abi)) ACMStream_fnRead(int*iface,int start,int
samples,int buffer){}
typedef struct _IAVIStreamImpl {
  int sInfo;
  int has;
} IAVIStreamImpl;
extern int __attribute__((ms_abi)) aso(void*);
int AVIFILE_OpenCompressor(IAVIStreamImpl *This)
{
  if (This->has != 0)
  aso(&This->has);
  sre(&This->sInfo);
}
--- CUT ---
This is some how related to the ms_abi attribute.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  GCC build triplet|x86_64-unknown-linux        |
   GCC host triplet|x86_64-unknown-linux        |
            Summary|[4.5/4.6 Regression] ICE in |[4.5/4.6 Regression] ICE in
                   |insert_save, at caller-     |insert_save with ms_abi
                   |save.c:1303                 |attribute


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

[Bug target/43662] [4.5/4.6 Regression] ICE in insert_save with ms_abi attribute

[rguenth at gcc dot gnu dot org]

-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P4


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

[Bug target/43662] [4.5/4.6 Regression] ICE in insert_save with ms_abi attribute

[hjl dot tools at gmail dot com]

------- Comment #4 from hjl dot tools at gmail dot com  2010-04-09 16:32 -------
It is caused by revision 157849:

http://gcc.gnu.org/ml/gcc-cvs/2010-03/msg00686.html


-- 

hjl dot tools at gmail dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jiez at gcc dot gnu dot org


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

[Bug target/43662] [4.5/4.6 Regression] ICE in insert_save with ms_abi attribute

[jiez at gcc dot gnu dot org]

------- Comment #5 from jiez at gcc dot gnu dot org  2010-04-10 15:29 -------
The patch:

http://gcc.gnu.org/ml/gcc-patches/2010-04/msg00466.html


-- 


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

[Bug target/43662] [4.5/4.6 Regression] ICE in insert_save with ms_abi attribute

[mikpe at it dot uu dot se]

------- Comment #6 from mikpe at it dot uu dot se  2010-04-14 13:20 -------
The patch causing this regression was backported to 4.4 in r157850, causing
current 4.4 to also regress.


-- 


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

[Bug target/43662] [4.4/4.5/4.6 Regression] ICE in insert_save with ms_abi attribute

[rguenth at gcc dot gnu dot org]

------- Comment #7 from rguenth at gcc dot gnu dot org  2010-04-14 14:08 -------
P1 for the regression on the release branch.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P4                          |P1
            Summary|[4.5/4.6 Regression] ICE in |[4.4/4.5/4.6 Regression] ICE
                   |insert_save with ms_abi     |in insert_save with ms_abi
                   |attribute                   |attribute


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

[Bug target/43662] [4.4/4.5/4.6 Regression] ICE in insert_save with ms_abi attribute

[rguenth at gcc dot gnu dot org]

-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.5.1                       |4.4.4


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

[Bug target/43662] [4.4/4.5/4.6 Regression] ICE in insert_save with ms_abi attribute

[mikpe at it dot uu dot se]

------- Comment #8 from mikpe at it dot uu dot se  2010-04-14 18:23 -------
(In reply to comment #5)
> The patch:
> 
> http://gcc.gnu.org/ml/gcc-patches/2010-04/msg00466.html

The test case in this patch breaks on 32-bit x86 because gcc complains that the
ms_abi attribute is only available in 64-bit mode.  The following fixes it:

--- gcc/testsuite/gcc.target/i386/pr43662.c.~1~
+++ gcc/testsuite/gcc.target/i386/pr43662.c
@@ -1,4 +1,5 @@
 /* { dg-do compile } */
+/* { dg-require-effective-target lp64 } */
 /* { dg-options "-O2" } */

 int __attribute__((ms_abi)) ACMStream_fnRead(int*iface,int start,int


-- 


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

[Bug target/43662] [4.4/4.5/4.6 Regression] ICE in insert_save with ms_abi attribute

[sezeroz at gmail dot com]

------- Comment #9 from sezeroz at gmail dot com  2010-04-14 18:59 -------
(In reply to comment #8)
> +/* { dg-require-effective-target lp64 } */

Adding llp64 to that would be helpful, too.


-- 


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

[Bug target/43662] [4.4/4.5/4.6 Regression] ICE in insert_save with ms_abi attribute

[jakub at gcc dot gnu dot org]

------- Comment #10 from jakub at gcc dot gnu dot org  2010-04-19 07:10 -------
I don't think llp64 there would be very much helpful, then all functions are
ms_abi and thus no ABI changes in between functions and no ICE.


-- 


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

[Bug target/43662] [4.4/4.5/4.6 Regression] ICE in insert_save with ms_abi attribute

[jiez at gcc dot gnu dot org]

------- Comment #11 from jiez at gcc dot gnu dot org  2010-04-19 07:24 -------
Thanks all. This is the updated patch:

http://gcc.gnu.org/ml/gcc-patches/2010-04/msg01157.html


-- 


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

[Bug target/43662] [4.4/4.5/4.6 Regression] ICE in insert_save with ms_abi attribute

[jiez at gcc dot gnu dot org]

------- Comment #12 from jiez at gcc dot gnu dot org  2010-04-19 10:03 -------
Subject: Bug 43662

Author: jiez
Date: Mon Apr 19 10:02:52 2010
New Revision: 158509

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=158509
Log:
        PR target/43662
        * reginfo.c (reinit_regs): Set caller_save_initialized_p
        to false.

        testsuite/
        PR target/43662
        * gcc.target/i386/pr43662.c: New test.

Added:
    trunk/gcc/testsuite/gcc.target/i386/pr43662.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/reginfo.c
    trunk/gcc/testsuite/ChangeLog


-- 


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

[Bug target/43662] [4.4/4.5/4.6 Regression] ICE in insert_save with ms_abi attribute

[jiez at gcc dot gnu dot org]

------- Comment #13 from jiez at gcc dot gnu dot org  2010-04-19 10:05 -------
Subject: Bug 43662

Author: jiez
Date: Mon Apr 19 10:04:43 2010
New Revision: 158510

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=158510
Log:
        PR target/43662
        * reginfo.c (reinit_regs): Set caller_save_initialized_p
        to false.

        testsuite/
        PR target/43662
        * gcc.target/i386/pr43662.c: New test.

Added:
    branches/gcc-4_5-branch/gcc/testsuite/gcc.target/i386/pr43662.c
Modified:
    branches/gcc-4_5-branch/gcc/ChangeLog
    branches/gcc-4_5-branch/gcc/reginfo.c
    branches/gcc-4_5-branch/gcc/testsuite/ChangeLog


-- 


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

[Bug target/43662] [4.4/4.5/4.6 Regression] ICE in insert_save with ms_abi attribute

[jiez at gcc dot gnu dot org]

------- Comment #14 from jiez at gcc dot gnu dot org  2010-04-19 10:06 -------
Subject: Bug 43662

Author: jiez
Date: Mon Apr 19 10:06:13 2010
New Revision: 158511

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=158511
Log:
        PR target/43662
        * reginfo.c (reinit_regs): Set caller_save_initialized_p
        to false.

        testsuite/
        PR target/43662
        * gcc.target/i386/pr43662.c: New test.

Added:
    branches/gcc-4_4-branch/gcc/testsuite/gcc.target/i386/pr43662.c
Modified:
    branches/gcc-4_4-branch/gcc/ChangeLog
    branches/gcc-4_4-branch/gcc/reginfo.c
    branches/gcc-4_4-branch/gcc/testsuite/ChangeLog


-- 


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

[Bug target/43662] [4.4/4.5/4.6 Regression] ICE in insert_save with ms_abi attribute

[jiez at gcc dot gnu dot org]

------- Comment #15 from jiez at gcc dot gnu dot org  2010-04-19 10:10 -------
Should be fixed now on trunk, 4.5 and 4.4 branches.


-- 


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

[Bug target/43662] [4.4/4.5/4.6 Regression] ICE in insert_save with ms_abi attribute

[jakub at gcc dot gnu dot org]

------- Comment #16 from jakub at gcc dot gnu dot org  2010-04-22 09:43 -------
Fixed.


-- 

jakub at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED


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

[Bug target/43662] [4.4/4.5/4.6 Regression] ICE in insert_save with ms_abi attribute

[pinskia at gcc dot gnu dot org]

------- Comment #17 from pinskia at gcc dot gnu dot org  2010-07-06 01:49 -------
*** Bug 44818 has been marked as a duplicate of this bug. ***


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mathieu dot pogeant at rdf
                   |                            |dot renesas dot com


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