[Bug other/49775] New: [4.6.1 Regression on AVR] ICE in based_loc_descr

[Bug other/49775] New: [4.6.1 Regression on AVR] ICE in based_loc_descr

[anitha.boyapati at atmel dot com]

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

           Summary: [4.6.1 Regression on AVR] ICE in based_loc_descr
           Product: gcc
           Version: unknown
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: other
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: anitha.boyapati@atmel.com


AVR target built with specs:

Using built-in specs.
COLLECT_GCC=avr-gcc
COLLECT_LTO_WRAPPER=/home/anitha/install/libexec/gcc/avr/4.6.1/lto-wrapper
Target: avr
Configured with: ../configure --prefix=/home/anitha/install --target=avr
--with-gmp=/proj/install/gmp-4.3.2/ --with-mpfr=/proj/install/mpfr-2.4.2/
--with-mpc=/proj/install/mpc-0.8.2/ --enable-languages=c,c++ --disable-libssp
--disable-nls --with-dwarf2
Thread model: single
gcc version 4.6.1 (GCC) 


The above compiler gives an ICE in based_loc_descr() of dwarf2out.c for 2
testcases present in gcc/testsuite/gcc.c-torture/compile/pr17906.c and
gcc/testsuite/gcc.c-torture/compile/pr35432.c.

Output for pr17906.c
--------------------

/home/anitha/patch-upgrade/gcc-4.6.1/gcc/testsuite/gcc.c-torture/compile/pr17906.c:
In function 'usc_recressive_func':
/home/anitha/patch-upgrade/gcc-4.6.1/gcc/testsuite/gcc.c-torture/compile/pr17906.c:7:1:
internal compiler error: in based_loc_descr, at dwarf2out.c:13563

Output for pr35432.c
--------------------

/gcc-4.6.1/gcc/testsuite/gcc.c-torture/compile/pr35432.c    (timeout = 300)
/home/anitha/patch-upgrade/gcc-4.6.1/gcc/testsuite/gcc.c-torture/compile/pr35432.c:
In function 'foo':
/home/anitha/patch-upgrade/gcc-4.6.1/gcc/testsuite/gcc.c-torture/compile/pr35432.c:11:1:
internal compiler error: in based_loc_descr, at dwarf2out.c:13563

[Bug other/49775] [4.6.1 Regression on AVR] ICE in based_loc_descr

[anitha.boyapati at atmel dot com]

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

--- Comment #1 from Anitha Boyapati <anitha.boyapati at atmel dot com> 2011-07-18 09:01:36 UTC ---

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

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

[Bug other/49775] [4.6.1 Regression on AVR] ICE in based_loc_descr

[gjl at gcc dot gnu.org]

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

Georg-Johann Lay <gjl at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |ice-on-valid-code
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2011.07.23 20:55:02
                 CC|                            |eric.weddington at atmel
                   |                            |dot com, gjl at gcc dot
                   |                            |gnu.org
   Target Milestone|---                         |4.6.2
     Ever Confirmed|0                           |1
      Known to fail|                            |4.7.0

--- Comment #2 from Georg-Johann Lay <gjl at gcc dot gnu.org> 2011-07-23 20:55:02 UTC ---
COnfirmed. I see these bugs in testsuite runs, too.

[Bug debug/49775] [4.6 Regression on AVR] ICE in based_loc_descr

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P4
            Summary|[4.6.1 Regression on AVR]   |[4.6 Regression on AVR] ICE
                   |ICE in based_loc_descr      |in based_loc_descr

[Bug debug/49775] [4.6 Regression on AVR] ICE in based_loc_descr

[anitha.boyapati at atmel dot com]

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

--- Comment #3 from Anitha Boyapati <anitha.boyapati at atmel dot com> 2011-08-02 05:54:03 UTC ---
I thought P3 should be OK. But P4 ...?

[Bug debug/49775] [4.6 Regression on AVR] ICE in based_loc_descr

[jakub at gcc dot gnu.org]

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

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

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

--- Comment #4 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-08-02 08:25:43 UTC ---
AVR isn't a primary nor secondary target, which implies P4/P5.  P3 is for not
yet categorized bugs, P1/P2 is for primary/secondary target affecting bugs
only.

[Bug debug/49775] [4.6 Regression on AVR] ICE in based_loc_descr

[gjl at gcc dot gnu.org]

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

--- Comment #5 from Georg-Johann Lay <gjl at gcc dot gnu.org> 2011-08-02 09:44:29 UTC ---
(In reply to comment #4)
> AVR isn't a primary nor secondary target, which implies P4/P5.  P3 is for not
> yet categorized bugs, P1/P2 is for primary/secondary target affecting bugs
> only.

Is there a documentation/description of the policy of that a target must be
capable of in order to be a secondary one?

[Bug debug/49775] [4.6 Regression on AVR] ICE in based_loc_descr

[jakub at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.6.2                       |4.6.3

--- Comment #6 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-10-26 17:13:46 UTC ---
GCC 4.6.2 is being released.

[Bug debug/49775] [4.6/4.7 Regression on AVR] ICE in based_loc_descr

[jakub at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.6.3                       |4.6.4

--- Comment #7 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-03-01 14:38:48 UTC ---
GCC 4.6.3 is being released.

[Bug debug/49775] [4.6/4.7/4.8 Regression on AVR] ICE in based_loc_descr

[praveen.r.kaushik at gmail dot com]

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

--- Comment #8 from Praveen Kaushik <praveen.r.kaushik at gmail dot com> 2012-09-21 10:36:20 UTC ---
This bug seems to be duplicate of this :

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


The assertion, 

gcc_assert ((SUPPORTS_STACK_ALIGNMENT
        && (elim == hard_frame_pointer_rtx
            || elim == stack_pointer_rtx))
       || elim == (frame_pointer_needed
                   ? hard_frame_pointer_rtx
                   : stack_pointer_rtx));

instead of being checked in 'compute_frame_pointer_to_fb_displacement ()' is
now assigned to 'frame_pointer_fb_offset_valid' and the assertion is moved to
'based_loc_descr()' (where it actually fails).

http://gcc.gnu.org/viewcvs/trunk/gcc/dwarf2out.c?r1=174715&r2=175018&diff_format=h


PR26504 was resolved by defining 'FRAME_POINTER_CFA_OFFSET' macro in avr.h.
This macro was removed later in Revision 171295, so that ARG_POINTER_CFA_OFFSET
can be defined.

These changes were done in order to fix PR48459.

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




snip from dwarf2out.c
----
#ifdef FRAME_POINTER_CFA_OFFSET
  reg = frame_pointer_rtx;
  offset += FRAME_POINTER_CFA_OFFSET (current_function_decl);
#else
  reg = arg_pointer_rtx;
  offset += ARG_POINTER_CFA_OFFSET (current_function_decl);
#endif

  elim = eliminate_regs (reg, VOIDmode, NULL_RTX);

----

Here `reg' is initialized with `arg_pointer_rtx' since FRAME_POINTER_CFA_OFFSET
is not defined.


----
frame_pointer_fb_offset_valid
    = ((SUPPORTS_STACK_ALIGNMENT
        && (elim == hard_frame_pointer_rtx
            || elim == stack_pointer_rtx))
       || elim == (frame_pointer_needed
                   ? hard_frame_pointer_rtx
                   : stack_pointer_rtx));
----

In this, `elim' is expected to be equal to `stack_pointer_rtx' (as
frame_pointer_needed is false) in order to have 'frame_pointer_fb_offset_valid'
value TRUE.


But AVR does not have any elimination to reduce `arg_pointer_rtx' to
`stack_pointer_rtx', so this assertion ought to fail in this case.


So IMO, to fix this issue either FRAME_POINTER_CFA_OFFSET should be defined or
an elimination should be defined which reduces `arg_pointer_rtx' to
`stack_pointer_rtx'.

[Bug debug/49775] [4.7/4.8/4.9 Regression on AVR] ICE in based_loc_descr

[jakub at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.6.4                       |4.7.4

--- Comment #9 from Jakub Jelinek <jakub at gcc dot gnu.org> 2013-04-12 15:16:08 UTC ---
GCC 4.6.4 has been released and the branch has been closed.

[Bug debug/49775] [4.7/4.8/4.9/4.10 Regression on AVR] ICE in based_loc_descr

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49775

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.7.4                       |4.8.4

--- Comment #10 from Richard Biener <rguenth at gcc dot gnu.org> ---
The 4.7 branch is being closed, moving target milestone to 4.8.4.

[Bug debug/49775] [4.8/4.9/5 Regression on AVR] ICE in based_loc_descr

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49775

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.8.4                       |4.8.5

--- Comment #11 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
GCC 4.8.4 has been released.

[Bug debug/49775] [4.8/4.9/5/6 Regression on AVR] ICE in based_loc_descr

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49775

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.8.5                       |4.9.3

--- Comment #12 from Richard Biener <rguenth at gcc dot gnu.org> ---
The gcc-4_8-branch is being closed, re-targeting regressions to 4.9.3.

[Bug debug/49775] [4.9/5/6 Regression on AVR] ICE in based_loc_descr

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49775

--- Comment #13 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
GCC 4.9.3 has been released.

[Bug debug/49775] [4.9/5/6 Regression on AVR] ICE in based_loc_descr

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49775

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.9.3                       |4.9.4