[Bug rtl-optimization/52175] [4.7 regression] ICE in maybe_record_trace_start after invalid dbr_schedule transformation

[Bug rtl-optimization/52175] [4.7 regression] ICE in maybe_record_trace_start after invalid dbr_schedule transformation

[rsandifo at gcc dot gnu.org]

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

rsandifo@gcc.gnu.org <rsandifo at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Target|                            |mips*-*-*
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2012-02-08
         AssignedTo|unassigned at gcc dot       |rsandifo at gcc dot gnu.org
                   |gnu.org                     |
     Ever Confirmed|0                           |1

--- Comment #1 from rsandifo at gcc dot gnu.org <rsandifo at gcc dot gnu.org> 2012-02-08 19:12:19 UTC ---
Testing a fix.

[Bug rtl-optimization/52175] New: [4.7 regression] ICE in maybe_record_trace_start after invalid dbr_schedule transformation

[rsandifo at gcc dot gnu.org]

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

             Bug #: 52175
           Summary: [4.7 regression] ICE in maybe_record_trace_start after
                    invalid dbr_schedule transformation
    Classification: Unclassified
           Product: gcc
           Version: 4.7.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: rtl-optimization
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: rsandifo@gcc.gnu.org


Created attachment 26618
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=26618
Testcase

The attached testcase fails on MIPS targets (e.g. mipsisa64-elf
and mips64-linux-gnu) when compiled with -O2 -g:

foo.c: In function ‘foo’:
foo.c:25:1: internal compiler error: in maybe_record_trace_start, at
dwarf2cfi.c:2231
Please submit a full bug report,
with preprocessed source if appropriate.
See <http://gcc.gnu.org/bugs.html> for instructions.

The problem is that dbr_schedule is applying its PLUS/MINUS
optimisation to a stack adjustment: that is, it fills a
delay slot with a stack adjustment that is only valid on
one edge, then inserts the opposite adjustment on the
other edge.

This is related to PR51471.  We shouldn't do this
adjustment for frame-related instructions.

[Bug rtl-optimization/52175] [4.7 regression] ICE in maybe_record_trace_start after invalid dbr_schedule transformation

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.7.0

[Bug rtl-optimization/52175] [4.7 regression] ICE in maybe_record_trace_start after invalid dbr_schedule transformation

[rsandifo at gcc dot gnu.org]

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

--- Comment #2 from rsandifo at gcc dot gnu.org <rsandifo at gcc dot gnu.org> 2012-02-11 09:00:47 UTC ---
Author: rsandifo
Date: Sat Feb 11 09:00:42 2012
New Revision: 184128

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=184128
Log:
gcc/
    PR rtl-optimization/52175
    * reorg.c (fill_slots_from_thread): Don't apply add/sub optimization
    to frame-related instructions.

gcc/testsuite/
    PR rtl-optimization/52175
    * gcc.c-torture/compile/pr52175.c: New test.

Added:
    trunk/gcc/testsuite/gcc.c-torture/compile/pr52175.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/reorg.c
    trunk/gcc/testsuite/ChangeLog

[Bug rtl-optimization/52175] [4.7 regression] ICE in maybe_record_trace_start after invalid dbr_schedule transformation

[rsandifo at gcc dot gnu.org]

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

rsandifo@gcc.gnu.org <rsandifo at gcc dot gnu.org> changed:

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

--- Comment #3 from rsandifo at gcc dot gnu.org <rsandifo at gcc dot gnu.org> 2012-02-11 09:16:06 UTC ---
Patch applied.