[Bug c++/55878] New: [4.7/4.8 Regression] --enable-checking=yes rejection of typeid

[Bug c++/55878] New: [4.7/4.8 Regression] --enable-checking=yes rejection of typeid

[jakub at gcc dot gnu.org]

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

             Bug #: 55878
           Summary: [4.7/4.8 Regression] --enable-checking=yes rejection
                    of typeid
    Classification: Unclassified
           Product: gcc
           Version: 4.8.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: jakub@gcc.gnu.org
                CC: jason@gcc.gnu.org


// { dg-do compile }
// { dg-options "-std=gnu++0x" }

#include <typeinfo>

struct S;

template <typename T>
static bool fn (S *s)
{
  return typeid (*s) == typeid (T);
}

struct S
{
};

bool x = fn<S> (__null);

is rejected with --enable-checking=yes compilers (but accepted with release
checking compilers), starting with
http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=173679

Is this code valid C++ or not, i.e. is the FE allowed to warn about incomplete
types even in the uninstantiated template?

[Bug c++/55878] [4.7/4.8 Regression] --enable-checking=yes rejection of typeid

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.7.3

[Bug c++/55878] [4.7/4.8 Regression] --enable-checking=yes rejection of typeid

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P1

--- Comment #1 from Richard Biener <rguenth at gcc dot gnu.org> 2013-01-07 15:48:24 UTC ---
Definitely P1, we should be consistent with accepting/rejecting code
with/without checking enabled.

[Bug c++/55878] [4.7/4.8 Regression] --enable-checking=yes rejection of typeid

[jason at gcc dot gnu.org]

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

--- Comment #2 from Jason Merrill <jason at gcc dot gnu.org> 2013-01-07 22:40:22 UTC ---
This is a bug that only appears with --enable-checking; I don't see how that
merits P1.

[Bug c++/55878] [4.7/4.8 Regression] --enable-checking=yes rejection of typeid

[jason at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2013-01-10
         AssignedTo|unassigned at gcc dot       |jason at gcc dot gnu.org
                   |gnu.org                     |
   Target Milestone|4.7.3                       |4.8.0
     Ever Confirmed|0                           |1

[Bug c++/55878] [4.7/4.8 Regression] --enable-checking=yes rejection of typeid

[jason at gcc dot gnu.org]

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

--- Comment #3 from Jason Merrill <jason at gcc dot gnu.org> 2013-01-10 02:54:43 UTC ---
Author: jason
Date: Thu Jan 10 02:54:39 2013
New Revision: 195075

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=195075
Log:
    PR c++/55878
    * rtti.c (build_typeid, get_typeid): Add complain parm.
    (get_tinfo_decl_dynamic): Likewise.
    * cp-tree.h, parser.c, pt.c: Adjust.

Added:
    trunk/gcc/testsuite/g++.dg/template/typeid1.C
Modified:
    trunk/gcc/cp/ChangeLog
    trunk/gcc/cp/cp-tree.h
    trunk/gcc/cp/parser.c
    trunk/gcc/cp/pt.c
    trunk/gcc/cp/rtti.c

[Bug c++/55878] [4.7/4.8 Regression] --enable-checking=yes rejection of typeid

[jason at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |rejects-valid
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED

--- Comment #4 from Jason Merrill <jason at gcc dot gnu.org> 2013-01-10 02:59:22 UTC ---
Fixed for 4.8.  No point fixing in 4.7 a bug that only comes up with
--enable-checking.