[Bug debug/60152] New: [4.9.0 Regression] multiple AT_calling_convention attributes generated after r205679

[Bug debug/60152] New: [4.9.0 Regression] multiple AT_calling_convention attributes generated after r205679

[dominiq at lps dot ens.fr]

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

            Bug ID: 60152
           Summary: [4.9.0 Regression] multiple  AT_calling_convention
                    attributes generated after r205679
           Product: gcc
           Version: 4.9.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: debug
          Assignee: unassigned at gcc dot gnu.org
          Reporter: dominiq at lps dot ens.fr
                CC: burnus at gcc dot gnu.org, rguenth at gcc dot gnu.org

After r205679, gfortran generates AT_calling_convention twice when compiling
gfortran.dg/namelist_51.f90 with '-g -O[01sg]', but only once with '-g -O2' and
above unless '-flto' is added. This causes the following waring on darwin

warning: invalid DWARF generated by the compiler: DIE 0x000001f0 has multiple 
AT_calling_convention attributes in '...'

[Bug debug/60152] [4.9 Regression] multiple AT_calling_convention attributes generated after r205679

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2014-02-12
                 CC|rguenth at gcc dot gnu.org         |
   Target Milestone|---                         |4.9.0
            Summary|[4.9.0 Regression] multiple |[4.9 Regression] multiple
                   | AT_calling_convention      |AT_calling_convention
                   |attributes generated after  |attributes generated after
                   |r205679                     |r205679
     Ever confirmed|0                           |1

--- Comment #1 from Richard Biener <rguenth at gcc dot gnu.org> ---
Confirmed.

[Bug debug/60152] [4.9 Regression] multiple AT_calling_convention attributes generated after r205679

[burnus at gcc dot gnu.org]

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

--- Comment #2 from Tobias Burnus <burnus at gcc dot gnu.org> ---
See PR 60060 comment 7 for some details and a backtrace of the two
add_calling_convention_attribute calls.

[Bug debug/60152] [4.9 Regression] multiple AT_calling_convention attributes generated after r205679

[jakub at gcc dot gnu.org]

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

--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Author: jakub
Date: Thu Feb 13 22:42:44 2014
New Revision: 207770

URL: http://gcc.gnu.org/viewcvs?rev=207770&root=gcc&view=rev
Log:
    PR debug/60152
    * dwarf2out.c (gen_subprogram_die): Don't call
    add_calling_convention_attribute if subr_die is old_die.

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

[Bug debug/60152] [4.9 Regression] multiple AT_calling_convention attributes generated after r205679

[jakub at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |jakub at gcc dot gnu.org
         Resolution|---                         |FIXED

--- Comment #4 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Fixed.