[Bug c++/47906] New: r170459 regresses g++.dg/abi/mangle39.C on *-apple-darwin*

[Bug c++/47906] New: r170459 regresses g++.dg/abi/mangle39.C on *-apple-darwin*

[howarth at nitro dot med.uc.edu]

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

           Summary: r170459 regresses g++.dg/abi/mangle39.C on
                    *-apple-darwin*
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: howarth@nitro.med.uc.edu


The commit...

Author: jason
Date: Thu Feb 24 04:56:26 2011
New Revision: 170459

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=170459
Log:
    * cp-tree.h (DECL_PARM_LEVEL): New.
    (struct lang_decl_parm): Add level field.
    * name-lookup.c (function_parm_depth): New fn.
    * name-lookup.h: Declare it.
    * parser.c (cp_parser_parameter_declaration_list): Use it.
    * mangle.c (struct globals): Add parm_depth field.
    (write_bare_function_type): Adjust it.
    (write_expression): Include the level delta in PARM_DECL mangling
    for abi >= 6.

    * semantics.c (finish_decltype_type): Remove shortcut for decltype
    of id-expression.
    * mangle.c (write_type) [DECLTYPE_TYPE]: Strip it here for abi < 6.

Added:
    trunk/gcc/testsuite/g++.dg/abi/mangle45.C
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/common.opt
    trunk/gcc/cp/ChangeLog
    trunk/gcc/cp/cp-tree.h
    trunk/gcc/cp/mangle.c
    trunk/gcc/cp/name-lookup.c
    trunk/gcc/cp/name-lookup.h
    trunk/gcc/cp/parser.c
    trunk/gcc/cp/semantics.c
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/g++.dg/abi/mangle39.C
    trunk/gcc/testsuite/g++.dg/cpp0x/trailing1.C
    trunk/gcc/testsuite/g++.dg/template/canon-type-12.C
    trunk/gcc/testsuite/g++.dg/template/canon-type-9.C

regresses g++.dg/abi/mangle39.C on both powerpc-apple-darwin9 and
x86_64-apple-darwin10...

FAIL: g++.dg/abi/mangle39.C scan-assembler _Z1gIiEvRK1AIT_EDTixfL0p_Li0EE

at -m32/-m64. It also introduces the new failures of...

FAIL: g++.dg/abi/mangle45.C scan-assembler _Z1fIiEvT_DtfL0p_E
FAIL: g++.dg/abi/mangle45.C scan-assembler _Z1gIiEvT_PFDtfL0p_EvE
FAIL: g++.dg/abi/mangle45.C scan-assembler _Z1kIiEvT_PFPAszfL0p__iPS0_E

for the new testcase.

[Bug c++/47906] r170459 regresses g++.dg/abi/mangle39.C on *-apple-darwin*

[jason at gcc dot gnu.org]

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

Jason Merrill <jason at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2011.02.27 08:15:47
                 CC|                            |jason at gcc dot gnu.org
     Ever Confirmed|0                           |1

--- Comment #1 from Jason Merrill <jason at gcc dot gnu.org> 2011-02-27 08:15:47 UTC ---
Ah, that would be because darwin doesn't support the ABI forward-compatibility
aliases.  I guess we should specify -fabi-version=6 for those tests.

[Bug c++/47906] r170459 regresses g++.dg/abi/mangle39.C on *-apple-darwin*

[jason at gcc dot gnu.org]

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

--- Comment #2 from Jason Merrill <jason at gcc dot gnu.org> 2011-02-27 14:39:17 UTC ---
Author: jason
Date: Sun Feb 27 14:39:15 2011
New Revision: 170543

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=170543
Log:
    PR c++/47906
    * g++.dg/abi/mangle39.C: Specify -fabi-version=6.
    * g++.dg/abi/mangle45.C: Specify -fabi-version=6.

Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/g++.dg/abi/mangle39.C
    trunk/gcc/testsuite/g++.dg/abi/mangle45.C

[Bug c++/47906] r170459 regresses g++.dg/abi/mangle39.C on *-apple-darwin*

[jason at gcc dot gnu.org]

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

Jason Merrill <jason at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |WAITING

--- Comment #3 from Jason Merrill <jason at gcc dot gnu.org> 2011-02-27 14:39:59 UTC ---
Do the tests pass on darwin now?

[Bug c++/47906] r170459 regresses g++.dg/abi/mangle39.C on *-apple-darwin*

[howarth at nitro dot med.uc.edu]

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

--- Comment #4 from Jack Howarth <howarth at nitro dot med.uc.edu> 2011-02-28 04:17:32 UTC ---
Yes the g++.dg/abi/mangle39.C and g++.dg/abi/mangle45.C now pass but it appears
we also need to do the same fix for these two failures as well...

FAIL: g++.dg/cpp0x/trailing1.C scan-assembler _Z1lIiEDtfp_ET_
FAIL: g++.dg/cpp0x/trailing1.C scan-assembler _Z1mIiLi1EEDtT0_ET_

[Bug c++/47906] r170459 regresses g++.dg/abi/mangle39.C on *-apple-darwin*

[jason at gcc dot gnu.org]

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

--- Comment #5 from Jason Merrill <jason at gcc dot gnu.org> 2011-02-28 06:55:11 UTC ---
Author: jason
Date: Mon Feb 28 06:55:08 2011
New Revision: 170556

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=170556
Log:
    PR c++/47906
    * g++.dg/cpp0x/trailing1.C: Specify -fabi-version=6.

Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/g++.dg/cpp0x/trailing1.C

[Bug c++/47906] r170459 regresses g++.dg/abi/mangle39.C on *-apple-darwin*

[paolo.carlini at oracle dot com]

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

Paolo Carlini <paolo.carlini at oracle dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|WAITING                     |RESOLVED
                 CC|jason at redhat dot com     |
         Resolution|                            |FIXED
   Target Milestone|---                         |4.6.0

--- Comment #6 from Paolo Carlini <paolo.carlini at oracle dot com> 2011-10-02 10:20:27 UTC ---
Fixed then.