[Bug c/39959] New: [4.5 Regression] IMA is broken

[Bug c/39959] New: [4.5 Regression] IMA is broken

[rguenth at gcc dot gnu dot org]

FAIL: gcc.dg/pr34989-1.c (internal compiler error)
FAIL: gcc.dg/pr34989-1.c (internal compiler error)
FAIL: gcc.dg/pr34989-1.c (test for excess errors)
FAIL: gcc.dg/pr34989-1.c (test for excess errors)

/space/rguenther/src/svn/trunk/gcc/testsuite/gcc.dg/pr34989-2.c: In function
'syslogd_main':^M
/space/rguenther/src/svn/trunk/gcc/testsuite/gcc.dg/pr34989-2.c:3: error:
non-trivial conversion at assignment^M
struct globals * const^M
struct globals *^M
# .MEM_3 = VDEF <.MEM_2(D)>^M
ptr_to_globals = 0B;^M


-- 
           Summary: [4.5 Regression] IMA is broken
           Product: gcc
           Version: 4.5.0
            Status: UNCONFIRMED
          Keywords: wrong-code
          Severity: normal
          Priority: P3
         Component: c
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: rguenth at gcc dot gnu dot org


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

[Bug c/39959] [4.5 Regression] IMA is broken

[rguenth at gcc dot gnu dot org]

-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.5.0


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

[Bug c/39959] [4.5 Regression] IMA is broken

[hp at gcc dot gnu dot org]

------- Comment #1 from hp at gcc dot gnu dot org  2009-04-30 00:16 -------
Noticed on cris-elf too, once the build issues were resolved.
Thus I don't have a better certain regression-range than 146694:146982, but
with a previous patch by Matz I didn't see this at 146828, so perhaps it's
146828:146982 (unless of course e.g. the regression is due to rtl-out-of-ssa
and that previous patch
<http://gcc.gnu.org/ml/gcc-patches/2009-04/msg02133.html> solved or covered
it).


-- 

hp at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hp at gcc dot gnu dot org
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2009-04-30 00:16:29
               date|                            |


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

[Bug c/39959] [4.5 Regression] IMA is broken

[hp at gcc dot gnu dot org]

------- Comment #2 from hp at gcc dot gnu dot org  2009-04-30 00:24 -------
I see what appears as the same bug for gcc.dg/struct/wo_prof_double_malloc.c
too, so I won't open a separate PR for that regression:

Running /tmp/hpautotest-gcc1/gcc/gcc/testsuite/gcc.dg/struct/struct-reorg.exp
...
XPASS: gcc.dg/struct/wo_prof_array_through_pointer.c scan-ipa-dump
ipa_struct_reorg "Number of structures to transform \
is 1"
FAIL: gcc.dg/struct/wo_prof_double_malloc.c (internal compiler error)
FAIL: gcc.dg/struct/wo_prof_double_malloc.c (test for excess errors)
WARNING: gcc.dg/struct/wo_prof_double_malloc.c compilation failed to produce
executable

gcc.log (^M's elided):

Executing on host: /tmp/hpautotest-gcc1/cris-elf/gccobj/gcc/xgcc
-B/tmp/hpautotest-gcc1/cris-elf/gccobj/gcc/
/tmp/hpautotest-gcc1/gcc/gcc/testsuite/gcc.dg/struct/wo_prof_double_malloc.c  
-O3 -fipa-struct-reorg -fdump-ipa-all -fwhole-program -combine
-fipa-type-escape   -isystem
/tmp/hpautotest-gcc1/cris-elf/gccobj/cris-elf/./newlib/targ-include -isystem
/tmp/hpautotest-gcc1/gcc/newlib/libc/include
-B/tmp/hpautotest-gcc1/cris-elf/gccobj/cris-elf/./libgloss/cris/
-L/tmp/hpautotest-gcc1/cris-elf/gccobj/cris-elf/./libgloss/cris
-L/tmp/hpautotest-gcc1/gcc/libgloss/cris 
-B/tmp/hpautotest-gcc1/cris-elf/gccobj/cris-elf/./newlib/
-L/tmp/hpautotest-gcc1/cris-elf/gccobj/cris-elf/./newlib -sim3  -lm   -o
./wo_prof_double_malloc.exe    (timeout = 300)
/tmp/hpautotest-gcc1/gcc/gcc/testsuite/gcc.dg/struct/wo_prof_double_malloc.c:
In function 'main':
/tmp/hpautotest-gcc1/gcc/gcc/testsuite/gcc.dg/struct/wo_prof_double_malloc.c:26:
error: non-trivial conversion at assignment
struct type_struct *
struct test_struct_sub.1 *
# .MEM_41 = VDEF <.MEM_18>
*D.2617_31 = D.2619_40;

/tmp/hpautotest-gcc1/gcc/gcc/testsuite/gcc.dg/struct/wo_prof_double_malloc.c:26:
error: non-trivial conversion at assignment
struct type_struct *
struct test_struct_sub.0 *
# .MEM_42 = VDEF <.MEM_41>
*D.2616_32 = D.2618_36;

/tmp/hpautotest-gcc1/gcc/gcc/testsuite/gcc.dg/struct/wo_prof_double_malloc.c:26:
internal compiler error: verify_stmts failed
Please submit a full bug report,


-- 


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

[Bug c/39959] [4.5 Regression] IMA is broken

[hp at gcc dot gnu dot org]

------- Comment #3 from hp at gcc dot gnu dot org  2009-04-30 01:24 -------
(In reply to comment #2)
> I see what appears as the same bug for gcc.dg/struct/wo_prof_double_malloc.c

Woops, ok, I noticed that's PR39960.  Ignore this and the previous comments. :)


-- 


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

[Bug c/39959] [4.5 Regression] IMA is broken

[mmitchel at gcc dot gnu dot org]

-- 

mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P1


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

[Bug c/39959] [4.5 Regression] IMA is broken

[janis at gcc dot gnu dot org]

------- Comment #4 from janis at gcc dot gnu dot org  2009-07-08 20:46 -------
On powerpc*-linux this test begins failing in the same way with this patch:

    http://gcc.gnu.org/viewcvs?view=rev&rev=146831

    r146831 | rguenth | 2009-04-27 11:18:38 +0000 (Mon, 27 Apr 2009)


-- 


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

[Bug c/39959] [4.5 Regression] IMA is broken

[janis at gcc dot gnu dot org]

------- Comment #5 from janis at gcc dot gnu dot org  2009-07-08 21:28 -------
The test started failing with the patch reported in comment #8 because it
enabled type checking; sorry for the noise.


-- 


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

[Bug c/39959] [4.5 Regression] IMA is broken

[rguenth at gcc dot gnu dot org]

------- Comment #6 from rguenth at gcc dot gnu dot org  2009-07-17 21:08 -------
I have a patch.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |rguenth at gcc dot gnu dot
                   |dot org                     |org
             Status|NEW                         |ASSIGNED
   Last reconfirmed|2009-04-30 00:16:29         |2009-07-17 21:08:55
               date|                            |


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

[Bug c/39959] [4.5 Regression] IMA is broken

[hjl dot tools at gmail dot com]

------- Comment #7 from hjl dot tools at gmail dot com  2009-08-03 14:41 -------
It was fixed between revision revision 150368 and revision 150371.


-- 


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

[Bug c/39959] [4.5 Regression] IMA is broken

[hp at gcc dot gnu dot org]

------- Comment #8 from hp at gcc dot gnu dot org  2009-08-16 00:16 -------
(In reply to comment #7)
> It was fixed between revision revision 150368 and revision 150371.

For the record, "covered up" rather than "fixed" according to
<http://gcc.gnu.org/ml/gcc-patches/2009-08/msg00102.html>.

Richi, the "I have a patch" comment, is that stale or still in the pipe?


-- 


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

[Bug c/39959] [4.5 Regression] IMA is broken

[rguenther at suse dot de]

------- Comment #9 from rguenther at suse dot de  2009-08-16 11:31 -------
Subject: Re:  [4.5 Regression] IMA is broken

On Sun, 16 Aug 2009, hp at gcc dot gnu dot org wrote:

> ------- Comment #8 from hp at gcc dot gnu dot org  2009-08-16 00:16 -------
> (In reply to comment #7)
> > It was fixed between revision revision 150368 and revision 150371.
> 
> For the record, "covered up" rather than "fixed" according to
> <http://gcc.gnu.org/ml/gcc-patches/2009-08/msg00102.html>.
> 
> Richi, the "I have a patch" comment, is that stale or still in the pipe?

Well, it's stale.  It broke too much.  The idea was to fix it up
during gimplification which we should be able to do now.

Richard.


-- 


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

[Bug c/39959] [4.5 Regression] IMA is broken

[pinskia at gcc dot gnu dot org]

------- Comment #10 from pinskia at gcc dot gnu dot org  2009-09-18 05:10 -------
*** Bug 41052 has been marked as a duplicate of this bug. ***


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |cthiel at cse dot unl dot
                   |                            |edu


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

[Bug c/39959] [4.5 Regression] IMA is broken

[pinskia at gcc dot gnu dot org]

------- Comment #11 from pinskia at gcc dot gnu dot org  2009-09-23 16:14 -------
*** Bug 41450 has been marked as a duplicate of this bug. ***


-- 


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

[Bug c/39959] [4.5 Regression] IMA is broken

[rguenth at gcc dot gnu dot org]

------- Comment #12 from rguenth at gcc dot gnu dot org  2009-09-23 16:18 -------
P1?  This is --combine being randomly broken with the C FEs inability to
properly
unify all same types from different TUs.  An ICE with checking only, but a
(non-regression) possible wrong-code issue due to aliasing effects (without
a testcase though).

P3 for re-consideration.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mmitchel at gcc dot gnu dot
                   |                            |org
           Keywords|                            |ice-checking, ice-on-valid-
                   |                            |code
           Priority|P1                          |P3


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

[Bug c/39959] [4.5 Regression] IMA is broken

[rguenth at gcc dot gnu dot org]

------- Comment #14 from rguenth at gcc dot gnu dot org  2009-10-14 21:28 -------
We also have

FAIL: gcc.dg/pr34668-1.c (internal compiler error)
FAIL: gcc.dg/pr34668-1.c (test for excess errors)


-- 


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

[Bug c/39959] [4.5 Regression] IMA is broken

[rguenth at gcc dot gnu dot org]

------- Comment #13 from rguenth at gcc dot gnu dot org  2009-10-14 21:28 -------
*** Bug 41086 has been marked as a duplicate of this bug. ***


-- 


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

[Bug c/39959] [4.5 Regression] IMA is broken

[pinskia at gcc dot gnu dot org]

------- Comment #15 from pinskia at gcc dot gnu dot org  2009-11-22 18:56 -------
*** Bug 42147 has been marked as a duplicate of this bug. ***


-- 

pinskia at gcc dot gnu dot org changed:

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


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

[Bug c/39959] [4.5 Regression] IMA is broken

[steven at gcc dot gnu dot org]

------- Comment #16 from steven at gcc dot gnu dot org  2010-01-03 23:08 -------
Can we close this bug as WONTFIX, and remove IMA for GCC 4.5?


-- 


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

[Bug c/39959] [4.5 Regression] IMA is broken

[joseph at codesourcery dot com]

------- Comment #17 from joseph at codesourcery dot com  2010-01-05 11:25 -------
Subject: Re:  [4.5 Regression] IMA is broken

See what I said in <http://gcc.gnu.org/ml/gcc/2009-09/msg00519.html> 
suggesting deprecating the implementation of -combine and keeping the 
interface.


-- 


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

[Bug c/39959] [4.5 Regression] IMA is broken

[rguenth at gcc dot gnu dot org]

------- Comment #18 from rguenth at gcc dot gnu dot org  2010-01-14 16:45 -------
Not working on this.  The ICE will go away with release checking.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|rguenth at gcc dot gnu dot  |unassigned at gcc dot gnu
                   |org                         |dot org
             Status|ASSIGNED                    |NEW
           Priority|P3                          |P2


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

[Bug c/39959] [4.5 Regression] IMA is broken

[pinskia at gcc dot gnu dot org]

------- Comment #19 from pinskia at gcc dot gnu dot org  2010-01-29 22:32 -------
*** Bug 42902 has been marked as a duplicate of this bug. ***


-- 

pinskia at gcc dot gnu dot org changed:

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


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

[Bug c/39959] [4.5 Regression] IMA is broken, gcc.dg/pr34668-1.c, gcc.dg/pr34668-2.c ICE

[ghazi at gcc dot gnu dot org]

------- Comment #20 from ghazi at gcc dot gnu dot org  2010-03-30 17:22 -------
Still have gcc.dg/pr34668-1.c failing on mainline (with checking enabled).


-- 

ghazi at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Last reconfirmed|2009-11-16 16:38:01         |2010-03-30 17:22:01
               date|                            |


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

[Bug c/39959] [4.5 Regression] IMA is broken, gcc.dg/pr34668-1.c, gcc.dg/pr34668-2.c ICE

[rguenth at gcc dot gnu dot org]

------- Comment #21 from rguenth at gcc dot gnu dot org  2010-04-06 11:19 -------
GCC 4.5.0 is being released.  Deferring to 4.5.1.


-- 

rguenth at gcc dot gnu dot org changed:

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


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

[Bug c/39959] [4.5/4.6 Regression] IMA is broken, gcc.dg/pr34668-1.c, gcc.dg/pr34668-2.c ICE

[pinskia at gcc dot gnu dot org]

------- Comment #22 from pinskia at gcc dot gnu dot org  2010-06-02 20:52 -------
*** Bug 44395 has been marked as a duplicate of this bug. ***


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |zeccav at gmail dot com


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

[Bug c/39959] [4.5/4.6 Regression] IMA is broken, gcc.dg/pr34668-1.c, gcc.dg/pr34668-2.c ICE

[hp at gcc dot gnu dot org]

------- Comment #23 from hp at gcc dot gnu dot org  2010-07-03 03:43 -------
This is fixed, isn't it?  No longer fails, c:a r161653 dudging from
gcc-testresults@.


-- 


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

[Bug c/39959] [4.5 Regression] IMA is broken, gcc.dg/pr34668-1.c, gcc.dg/pr34668-2.c ICE

[rguenth at gcc dot gnu dot org]

------- Comment #24 from rguenth at gcc dot gnu dot org  2010-07-03 10:50 -------
Fixed in 4.6, wontfix in 4.5.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
      Known to fail|                            |4.5.1
         Resolution|                            |FIXED
            Summary|[4.5/4.6 Regression] IMA is |[4.5 Regression] IMA is
                   |broken, gcc.dg/pr34668-1.c, |broken, gcc.dg/pr34668-1.c,
                   |gcc.dg/pr34668-2.c ICE      |gcc.dg/pr34668-2.c ICE
   Target Milestone|4.5.1                       |4.6.0


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