[Bug rtl-optimization/47620] New: [4.6 Regression] Profiledbootstrap failure on powerpc-linux

[Bug rtl-optimization/47620] New: [4.6 Regression] Profiledbootstrap failure on powerpc-linux

[jakub at gcc dot gnu.org]

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

           Summary: [4.6 Regression] Profiledbootstrap failure on
                    powerpc-linux
           Product: gcc
           Version: unknown
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: rtl-optimization
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: jakub@gcc.gnu.org
                CC: aoliva@gcc.gnu.org
            Target: powerpc64-linux


Created attachment 23255
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=23255
ice.i

32-bit powerpc64-linux --with-cpu=default32 bootstrap on the trunk fails with
ICE in queue_insn, suspect change are the recent -fcompare-debug scheduler
changes.

./cc1 -m32 -g -O2 ice.i -quiet -fprofile-use
../../gcc/cfganal.c: In function ‘find_unreachable_blocks’:
../../gcc/cfganal.c:327:1: internal compiler error: in queue_insn, at
haifa-sched.c:1322
Please submit a full bug report,
with preprocessed source if appropriate.
See <http://gcc.gnu.org/bugs.html> for instructions.

can be reproduced even with cross-compiler.

[Bug rtl-optimization/47620] [4.6 Regression] Profiledbootstrap failure on powerpc-linux

[jakub at gcc dot gnu.org]

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

--- Comment #1 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-02-06 13:02:56 UTC ---
Created attachment 23256
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=23256
ice.gcda

The corresponding *.gcda file.

[Bug rtl-optimization/47620] [4.6 Regression] Profiledbootstrap failure on powerpc-linux

[jakub at gcc dot gnu.org]

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

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.6.0

--- Comment #2 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-02-06 13:06:32 UTC ---
Doesn't fail with ./cc1 -m32 -O2 ice.i -quiet -fprofile-use
The ICE is because queue_insn is called with a DEBUG_INSN.

[Bug rtl-optimization/47620] [4.6 Regression] Profiledbootstrap failure on powerpc-linux

[jakub at gcc dot gnu.org]

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

--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-02-07 09:04:52 UTC ---
Reverting the r169513 change fixes the ICE.  To reproduce, just a
--target=powerpc64-linux --enable-languages=c cross is enough, no need to
bother with cross-binutils etc.

Alex, could you please have a look at this?

[Bug rtl-optimization/47620] [4.6 Regression] Profiledbootstrap failure on powerpc-linux

[jakub at gcc dot gnu.org]

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

--- Comment #4 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-02-07 13:45:18 UTC ---
Also seen on s390x:
https://bugzilla.redhat.com/show_bug.cgi?id=675711

[Bug rtl-optimization/47620] [4.6 Regression] Profiledbootstrap failure on powerpc-linux

[rguenth at gcc dot gnu.org]

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

Richard Guenther <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Target|powerpc64-linux             |powerpc64-linux,
                   |                            |s390x-linux
           Priority|P3                          |P1
            Version|unknown                     |4.6.0

[Bug debug/47620] [4.6 Regression] Profiledbootstrap failure on powerpc-linux

[aoliva at gcc dot gnu.org]

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

Alexandre Oliva <aoliva at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2011.02.12 12:49:17
          Component|rtl-optimization            |debug
     Ever Confirmed|0                           |1

--- Comment #5 from Alexandre Oliva <aoliva at gcc dot gnu.org> 2011-02-12 12:49:17 UTC ---
Mine

[Bug debug/47620] [4.6 Regression] Profiledbootstrap failure on powerpc-linux

[aoliva at gcc dot gnu.org]

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

--- Comment #6 from Alexandre Oliva <aoliva at gcc dot gnu.org> 2011-02-12 13:05:10 UTC ---
Created attachment 23317
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=23317
Patch that fixes the bug

Here's a patch that fixes the problem.  Jakub, is it easy for you to test it on
the affected platforms?  Please? :-)  TIA,

[Bug debug/47620] [4.6 Regression] Profiledbootstrap failure on powerpc-linux

[aoliva at gcc dot gnu.org]

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

Alexandre Oliva <aoliva at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ro at gcc dot gnu.org

--- Comment #7 from Alexandre Oliva <aoliva at gcc dot gnu.org> 2011-02-12 13:49:55 UTC ---
*** Bug 47630 has been marked as a duplicate of this bug. ***

[Bug debug/47620] [4.6 Regression] Profiledbootstrap failure on powerpc-linux

[jakub at gcc dot gnu.org]

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

--- Comment #8 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-02-13 15:57:05 UTC ---
(In reply to comment #6)
> Here's a patch that fixes the problem.  Jakub, is it easy for you to test it on
> the affected platforms?  Please? :-)  TIA,

Profiledootstrapped/regtested on x86_64-linux, i686-linux and powerpc64-linux
(64-bit), profiledbootstrapped so far on powerpc-linux (regtest still pending).

[Bug debug/47620] [4.6 Regression] Profiledbootstrap failure on powerpc-linux

[dan at danny dot cz]

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

Dan Horak <dan at danny dot cz> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |dan at danny dot cz

--- Comment #9 from Dan Horak <dan at danny dot cz> 2011-02-14 12:54:59 UTC ---
build of gcc-4.6.0-0.7.fc15 fails on s390, s390x looks good this time.

gcc-4.6.0-0.5.fc15 is in the buildroot, but the problem is in the freshly built
gcc and not in the one from buildroot if I understand it correctly.

for build logs please see
http://s390.koji.fedoraproject.org/koji/taskinfo?taskID=302436

[Bug debug/47620] [4.6 Regression] Profiledbootstrap failure on powerpc-linux

[jakub at gcc dot gnu.org]

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

--- Comment #10 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-02-16 23:49:56 UTC ---
Created attachment 23371
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=23371
ice.i.bz2

For the s390-linux ICE, which is still present on the current trunk:
bunzip2 ice.i.bz2
../configure --target s390-linux --enable-languages=c
gcc/cc1 -O2 -g -m31 -fprofile-use -march=z9-109 -mtune=z10 ice.i

[Bug debug/47620] [4.6 Regression] Profiledbootstrap failure on powerpc-linux

[jakub at gcc dot gnu.org]

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

--- Comment #11 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-02-16 23:51:11 UTC ---
Created attachment 23372
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=23372
ice.gcda

Corresponding ice.gcda.

[Bug debug/47620] [4.6 Regression] Profiledbootstrap failure on powerpc-linux

[jakub at gcc dot gnu.org]

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

--- Comment #12 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-02-16 23:54:42 UTC ---
The ICE happens even with the #c6 patch applied on top of today's trunk.

[Bug debug/47620] [4.6 Regression] Profiledbootstrap failure on powerpc-linux

[jakub at gcc dot gnu.org]

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

--- Comment #13 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-02-17 08:18:38 UTC ---
On s390x-linux the profiledbootstrap actually fails the same way too apparently
(again, with #c6 patch applied).

[Bug debug/47620] [4.6 Regression] Profiledbootstrap failure on powerpc-linux

[aoliva at gcc dot gnu.org]

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

Alexandre Oliva <aoliva at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #23317|0                           |1
        is obsolete|                            |

--- Comment #14 from Alexandre Oliva <aoliva at gcc dot gnu.org> 2011-02-17 21:33:45 UTC ---
Created attachment 23390
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=23390
Patch that kills the bug harder

It turns out that deps from other blocks wouldn't always have tick 0, and debug
insns other than the first could have deps other than their immediate preds,
that would cause them to be queue for a later tick as well.  But then, why
bother?  If all dependencies of a debug insns are met, just add it to the ready
set, without even computing the tick it would get from its deps.  Like this
patch, that supersedes the earlier on.  Jakub, can you give it a spin on the
affected platforms, please?  I'm bootstrapping it on x86_64-linux-gnu and
i686-linux-gnu now.

[Bug debug/47620] [4.6 Regression] Profiledbootstrap failure on powerpc-linux

[jakub at gcc dot gnu.org]

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

--- Comment #15 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-02-18 10:48:25 UTC ---
(In reply to comment #14)
> Created attachment 23390 [details]
> Patch that kills the bug harder

While with the previous patch gcc didn't profiledbootstrap on s390-linux, nor
apparently s390x-linux and ppc-linux too (yet bootstrap went fine),
with this patch instead of the previous one gcc successfully
profiledbootstrapped
on {x86_64,i686,ppc,ppc64,s390,s390x}-linux with no regressions against the
bootstrap regtest results without that patch.

[Bug debug/47620] [4.6 Regression] Profiledbootstrap failure on powerpc-linux

[jakub at gcc dot gnu.org]

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

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |J.J.vanderHeijden at gmail
                   |                            |dot com

--- Comment #16 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-02-19 07:46:47 UTC ---
*** Bug 47512 has been marked as a duplicate of this bug. ***

[Bug debug/47620] [4.6 Regression] Profiledbootstrap failure on powerpc-linux

[aoliva at gcc dot gnu.org]

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

--- Comment #17 from Alexandre Oliva <aoliva at gcc dot gnu.org> 2011-02-19 20:59:27 UTC ---
Author: aoliva
Date: Sat Feb 19 20:59:23 2011
New Revision: 170324

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=170324
Log:
PR debug/47620
PR debug/47630
* haifa-sched.c (fix_tick_ready): Skip tick computation
for debug insns.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/haifa-sched.c

[Bug debug/47620] [4.6 Regression] Profiledbootstrap failure on powerpc-linux

[aoliva at gcc dot gnu.org]

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

Alexandre Oliva <aoliva at gcc dot gnu.org> changed:

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

--- Comment #18 from Alexandre Oliva <aoliva at gcc dot gnu.org> 2011-02-19 21:25:23 UTC ---
Fixed