[Bug middle-end/48464] New: Regression @171649: ICE in setup_pressure_classes, at ira.c:877

[Bug middle-end/48464] New: Regression @171649: ICE in setup_pressure_classes, at ira.c:877

[jbglaw@lug-owl.de]

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

           Summary: Regression @171649: ICE in setup_pressure_classes, at
                    ira.c:877
           Product: gcc
           Version: 4.7.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: middle-end
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: jbglaw@lug-owl.de


Hi!

The large IRA merge (=66d9a7b9e8a0c85a51467330f51ead74e77b7eb8 in GIT) broke
the vax-linux target:

jbglaw@jbarbeitsplatz:/mnt/nfs-programming/toolchain/gcc-build/gcc$ ./xgcc -B .
-c foo.c
foo.c:1:0: internal compiler error: in setup_pressure_classes, at ira.c:877
Please submit a full bug report,
[...]


jbglaw@jbarbeitsplatz:/mnt/nfs-programming/toolchain/gcc-build/gcc$ cat foo.c   
extern int xx (int a, int b);
int
blub (double boo, int oo)           
{
long x;
x = boo * oo;
x += xx (boo, oo);
return x;
}

This is just configured with ../gcc/configure --target=vax-linux
--enable-languages=c --disable-threads on a i686-linux host/build system.

[Bug middle-end/48464] Regression @171649: ICE in setup_pressure_classes, at ira.c:877

[jbglaw@lug-owl.de]

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

Jan-Benedict Glaw <jbglaw@lug-owl.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Target|                            |vax-linux
                 CC|                            |jbglaw@lug-owl.de, vmakarov
                   |                            |at redhat dot com
               Host|                            |i686-linux
              Build|                            |i686-linux

--- Comment #1 from Jan-Benedict Glaw <jbglaw@lug-owl.de> 2011-04-05 16:07:47 UTC ---
Hint for VCS users (as bugzilla only wants to see released version names).

Known to work: r171648 = 9f50fa3742ab7c20e3e8ceb48b26c920b22ecd62
Known to fail: r171649 = 66d9a7b9e8a0c85a51467330f51ead74e7

[Bug middle-end/48464] [4.7 Regression] @171649: ICE in setup_pressure_classes, at ira.c:877

[pinskia at gcc dot gnu.org]

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

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |ice-on-valid-code
   Target Milestone|---                         |4.7.0
            Summary|Regression @171649: ICE in  |[4.7 Regression] @171649:
                   |setup_pressure_classes, at  |ICE in
                   |ira.c:877                   |setup_pressure_classes, at
                   |                            |ira.c:877

[Bug middle-end/48464] [4.7 Regression] @171649: ICE in setup_pressure_classes, at ira.c:877

[jbglaw@lug-owl.de]

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

--- Comment #2 from Jan-Benedict Glaw <jbglaw@lug-owl.de> 2011-04-09 20:58:00 UTC ---
Just to mention: This is not an ICE that shows up on this specific code
snipplet, but it is rather a start-up problem that either got merged in with
the IRA update or was uncovered by it:

vax-linux-gcc -c -x c /dev/null
/dev/null:1:0: internal compiler error: in setup_pressure_classes, at ira.c:877
Please submit a full bug report,
with preprocessed source if appropriate.
See <http://gcc.gnu.org/bugs.html> for instructions.

(This is r172231 = d82cf2b25062950d .)

[Bug middle-end/48464] [4.7 Regression] @171649: ICE in setup_pressure_classes, at ira.c:877

[vmakarov at redhat dot com]

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

--- Comment #3 from Vladimir Makarov <vmakarov at redhat dot com> 2011-04-11 18:44:59 UTC ---
There is typo in a loop condition resulting in taking hard registers of
LIM_REG_CLASS which happens a garbage for VAX.

I'll send a patch soon.

[Bug middle-end/48464] [4.7 Regression] @171649: ICE in setup_pressure_classes, at ira.c:877

[vmakarov at gcc dot gnu.org]

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

--- Comment #4 from Vladimir Makarov <vmakarov at gcc dot gnu.org> 2011-04-11 19:23:17 UTC ---
Author: vmakarov
Date: Mon Apr 11 19:23:13 2011
New Revision: 172274

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=172274
Log:
2011-04-11  Vladimir Makarov  <vmakarov@redhat.com>

    PR middle-end/48464
    * ira.c (setup_pressure_classes): Fix typo in loop condition.
    (setup_allocno_and_important_classes): Ditto.


Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/ira.c

[Bug middle-end/48464] [4.7 Regression] @171649: ICE in setup_pressure_classes, at ira.c:877

[jbglaw@lug-owl.de]

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

--- Comment #5 from Jan-Benedict Glaw <jbglaw@lug-owl.de> 2011-04-11 22:04:12 UTC ---
(In reply to comment #4)
> URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=172274
> Log:
> 2011-04-11  Vladimir Makarov  <vmakarov@redhat.com>
> 
>     PR middle-end/48464
>     * ira.c (setup_pressure_classes): Fix typo in loop condition.
>     (setup_allocno_and_important_classes): Ditto.

I updated to r172276 = 850ff64c7c083, but gcc still ICEs at the same line of
code. So there must be another root cause. However, at least one bug fixed,
good work :)

[Bug middle-end/48464] [4.7 Regression] @171649: ICE in setup_pressure_classes, at ira.c:877

[jbglaw@lug-owl.de]

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

--- Comment #6 from Jan-Benedict Glaw <jbglaw@lug-owl.de> 2011-05-27 13:49:51 UTC ---
It seems r174123 (= 59813b406b20d7b) not only fixed PR rtl-optimization/48971,
but also this issue:

    2011-05-13  Vladimir Makarov  <vmakarov@redhat.com>

        PR rtl-optimization/48971
        * ira.c (setup_pressure_classes): Don't check register move cost
        for classes with one registers.  Don't add pressure class if there
        is a pressure class with the same available hard registers.
        Check contains_reg_of_mode.  Fix a typo in collecting
        temp_hard_regset.  Ignore hard registers not belonging to a class.


Since this patch, this problem is gone.

Thanks!

[Bug middle-end/48464] [4.7 Regression] @171649: ICE in setup_pressure_classes, at ira.c:877

[jbglaw@lug-owl.de]

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

Jan-Benedict Glaw <jbglaw@lug-owl.de> changed:

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

--- Comment #7 from Jan-Benedict Glaw <jbglaw@lug-owl.de> 2011-05-27 14:09:30 UTC ---
Since this now works for me, I'm closing the bug.