[Bug bootstrap/48971] New: sparc-rtems does not build

[Bug bootstrap/48971] New: sparc-rtems does not build

[joel at gcc dot gnu.org]

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

           Summary: sparc-rtems does not build
           Product: gcc
           Version: 4.7.0
            Status: UNCONFIRMED
          Severity: critical
          Priority: P3
         Component: bootstrap
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: joel@gcc.gnu.org


As of yesterday's SVN, gcc is very broken for sparc-rtems.  The generated gcc
can't even compile a simple autoconf probe.  I would expect that sparc-elf is
similarly broken.  This was built with a native built from the same source. 
That reports

gcc (GCC) 4.7.0 20110510 (experimental) [trunk revision 173623]


This is from config.log.  Let me know what else I can provide.

configure:3246: checking for suffix of object files
configure:3268: /home2/joel/build/b-sparc-gcc/./gcc/xgcc
-B/home2/joel/build/b-sparc-gcc/./gcc/ -nostdinc
-B/home2/joel/build/b-sparc-gcc/sparc-rtems4.11/soft/newlib/ -isystem
/home2/joel/build/b-sparc-gcc/sparc-rtems4.11/soft/newlib/targ-include -isystem
/users/joel/test-gcc/gcc-svn/newlib/libc/include
-B/home2/joel/build/b-sparc-gcc/sparc-rtems4.11/soft/libgloss/sparc
-L/home2/joel/build/b-sparc-gcc/sparc-rtems4.11/soft/libgloss/libnosys
-L/users/joel/test-gcc/gcc-svn/libgloss/sparc
-B/users/joel/test-gcc/install-svn/sparc-rtems4.11/bin/
-B/users/joel/test-gcc/install-svn/sparc-rtems4.11/lib/ -isystem
/users/joel/test-gcc/install-svn/sparc-rtems4.11/include -isystem
/users/joel/test-gcc/install-svn/sparc-rtems4.11/sys-include  -msoft-float -c
-g -O2  conftest.c >&5
conftest.c: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.
configure:3272: $? = 1
configure: failed program was:
| /* confdefs.h */
| #define PACKAGE_NAME "GNU C Runtime Library"
| #define PACKAGE_TARNAME "libgcc"
| #define PACKAGE_VERSION "1.0"
| #define PACKAGE_STRING "GNU C Runtime Library 1.0"
| #define PACKAGE_BUGREPORT ""
| #define PACKAGE_URL "http://www.gnu.org/software/libgcc/"
| /* end confdefs.h.  */
|
| int
| main ()
| {
|
|   ;
|   return 0;
| }
configure:3286: error: in
`/home2/joel/build/b-sparc-gcc/sparc-rtems4.11/soft/libgcc':
configure:3289: error: cannot compute suffix of object files: cannot compile
See `config.log' for more details.

[Bug rtl-optimization/48971] [4.7 regression] ICE with -msoft-float -O2

[ebotcazou at gcc dot gnu.org]

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

Eric Botcazou <ebotcazou at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Target|                            |sparc-*-*
             Status|UNCONFIRMED                 |NEW
           Keywords|                            |ice-on-valid-code
   Last reconfirmed|                            |2011.05.12 06:56:03
          Component|bootstrap                   |rtl-optimization
                 CC|                            |ebotcazou at gcc dot
                   |                            |gnu.org
     Ever Confirmed|0                           |1
            Summary|sparc-rtems does not build  |[4.7 regression] ICE with
                   |                            |-msoft-float -O2
           Severity|critical                    |normal

--- Comment #1 from Eric Botcazou <ebotcazou at gcc dot gnu.org> 2011-05-12 06:56:03 UTC ---
This is -msoft-float though.  Again recent IRA changes I'm afraid...

[Bug rtl-optimization/48971] [4.7 regression] ICE with -msoft-float -O2

[law at redhat dot com]

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

Jeffrey A. Law <law at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |law at redhat dot com,
                   |                            |vmakarov at redhat dot com

--- Comment #2 from Jeffrey A. Law <law at redhat dot com> 2011-05-12 15:14:54 UTC ---
Vlad, this is an abort in setup_pressure_classes which apparently is totally
broken for sparc -msoft-float.

[Bug rtl-optimization/48971] [4.7 regression] ICE with -msoft-float -O2

[vmakarov at redhat dot com]

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

--- Comment #3 from Vladimir Makarov <vmakarov at redhat dot com> 2011-05-12 17:57:58 UTC ---
(In reply to comment #2)
> Vlad, this is an abort in setup_pressure_classes which apparently is totally
> broken for sparc -msoft-float.


I found the wrong code.  It is pretty simple but I need to check a few
platforms because the fix might affect other platform builds.

I hope I'll send the patch at the end of the day.

[Bug rtl-optimization/48971] [4.7 regression] ICE with -msoft-float -O2

[vmakarov at redhat dot com]

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

--- Comment #4 from Vladimir Makarov <vmakarov at redhat dot com> 2011-05-13 15:34:39 UTC ---
(In reply to comment #3)
> (In reply to comment #2)
> > Vlad, this is an abort in setup_pressure_classes which apparently is totally
> > broken for sparc -msoft-float.
> 
> 
> I found the wrong code.  It is pretty simple but I need to check a few
> platforms because the fix might affect other platform builds.
> 
> I hope I'll send the patch at the end of the day.

SPARC ICC register presents in ALL_REGS class only which can not be a pressure
class. That is the reason for the problem.  I also found a typo in the check
code (it collected hard registers of all non-pressure classes although it
should collect the pressure classes hard registers).  I found more complication
with the check code in MIPS target.  So it took more time than I did. 
Currently I am testing the patch and submit it for approval soon.

[Bug rtl-optimization/48971] [4.7 regression] ICE with -msoft-float -O2

[vmakarov at gcc dot gnu.org]

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

--- Comment #5 from Vladimir Makarov <vmakarov at gcc dot gnu.org> 2011-05-24 16:14:56 UTC ---
Author: vmakarov
Date: Tue May 24 16:14:51 2011
New Revision: 174123

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=174123
Log:
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.


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

[Bug rtl-optimization/48971] [4.7 regression] ICE with -msoft-float -O2

[ebotcazou at gcc dot gnu.org]

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

Eric Botcazou <ebotcazou at gcc dot gnu.org> changed:

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

--- Comment #6 from Eric Botcazou <ebotcazou at gcc dot gnu.org> 2011-05-25 07:59:16 UTC ---
This should work again now.