[Bug c++/51414] New: Broken diagnostic with invalid use of __underlying_type

[Bug c++/51414] New: Broken diagnostic with invalid use of __underlying_type

[reichelt at gcc dot gnu.org]

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

             Bug #: 51414
           Summary: Broken diagnostic with invalid use of
                    __underlying_type
    Classification: Unclassified
           Product: gcc
           Version: 4.7.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: reichelt@gcc.gnu.org


A broken diagnostic is generated on trunk for the following invalid code
snippet:

==============================
__underlying_type(int[1]) i;
==============================

bug.cc:1:25: error: '#'array_type' not supported by dump_expr#<expression
error>' is not an enumeration type
bug.cc:1:28: error: invalid type in declaration before ';' token

[Bug c++/51414] Broken diagnostic with invalid use of __underlying_type

[manu at gcc dot gnu.org]

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

Manuel López-Ibáñez <manu at gcc dot gnu.org> changed:

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

--- Comment #1 from Manuel López-Ibáñez <manu at gcc dot gnu.org> 2011-12-04 23:07:46 UTC ---
Clang 3.0:

pr51414.cc:1:1: error: only enumeration types have underlying types
__underlying_type(int[1]) i;
^
1 error generated.

[Bug c++/51414] Broken diagnostic with invalid use of __underlying_type

[paolo.carlini at oracle dot com]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2011-12-05
         AssignedTo|unassigned at gcc dot       |paolo.carlini at oracle dot
                   |gnu.org                     |com
     Ever Confirmed|0                           |1

--- Comment #2 from Paolo Carlini <paolo.carlini at oracle dot com> 2011-12-05 10:46:00 UTC ---
Seems just a trivial thinko if mine: %qE instead of %qT.

[Bug c++/51414] Broken diagnostic with invalid use of __underlying_type

[paolo at gcc dot gnu.org]

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

--- Comment #3 from paolo at gcc dot gnu.org <paolo at gcc dot gnu.org> 2011-12-05 18:21:21 UTC ---
Author: paolo
Date: Mon Dec  5 18:21:18 2011
New Revision: 182017

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=182017
Log:
/cp
2011-12-05  Paolo Carlini  <paolo.carlini@oracle.com>

    PR c++/51414
    * semantics.c (finish_underlying_type): Use %qT, not %qE for the
    error message.

/testsuite
2011-12-05  Paolo Carlini  <paolo.carlini@oracle.com>

    PR c++/51414
    * g++.dg/ext/underlying_type11.C: New.


Added:
    trunk/gcc/testsuite/g++.dg/ext/underlying_type11.C
Modified:
    trunk/gcc/cp/ChangeLog
    trunk/gcc/cp/semantics.c
    trunk/gcc/testsuite/ChangeLog

[Bug c++/51414] Broken diagnostic with invalid use of __underlying_type

[paolo.carlini at oracle dot com]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|---                         |4.7.0

--- Comment #4 from Paolo Carlini <paolo.carlini at oracle dot com> 2011-12-05 18:22:52 UTC ---
Fixed.