public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
From: "jason at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org> To: gcc-bugs@gcc.gnu.org Subject: [Bug c++/30195] Using declaration doesn't work in template. Date: Mon, 14 Nov 2011 17:59:00 -0000 [thread overview] Message-ID: <bug-30195-4-SM6T6SOCUM@http.gcc.gnu.org/bugzilla/> (raw) In-Reply-To: <bug-30195-4@http.gcc.gnu.org/bugzilla/> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=30195 --- Comment #5 from Jason Merrill <jason at gcc dot gnu.org> 2011-11-14 17:56:04 UTC --- Author: jason Date: Mon Nov 14 17:55:57 2011 New Revision: 181359 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=181359 Log: PR c++/6936 PR c++/25994 PR c++/26256 PR c++/30195 * search.c (lookup_field_1): Look through USING_DECL. (lookup_field_r): Call lookup_fnfields_slot instead of lookup_fnfields_1. * semantics.c (finish_member_declaration): Remove the check that prevents USING_DECLs from being verified by pushdecl_class_level. Call add_method for using declarations that designates functions if the using declaration is in a template class. Set DECL_IGNORED_P on class-scope using declarations. * typeck.c (build_class_member_access_expr): Handle USING_DECLs. * class.c (check_field_decls): Keep using declarations. (add_method): Remove two diagnostics about conflicting using declarations. * parser.c (cp_parser_nonclass_name): Handle USING_DECLs. * decl.c (start_enum): Call xref_tag whenever possible. * cp-tree.h (strip_using_decl): Declare, and reident the previous function. * name-lookup.c (strip_using_decl): New function. (supplement_binding_1): Call strip_using_decl on decl and bval. Perform most of the checks with USING_DECLs stripped. Also check that the target decl and the target bval does not refer to the same declaration. Allow pushing an enum multiple times in a template class. Adjustment to diagnose using redeclarations. Call diagnose_name_conflict. (push_class_level_binding): Call strip_using_decl on decl and bval. Perform most of the checks with USING_DECLs stripped. Return true if both decl and bval refer to USING_DECLs and are dependent. (diagnose_name_conflict): New function. Added: trunk/gcc/testsuite/g++.dg/cpp0x/forw_enum10.C trunk/gcc/testsuite/g++.dg/debug/using4.C trunk/gcc/testsuite/g++.dg/debug/using5.C trunk/gcc/testsuite/g++.dg/lookup/pr6936.C trunk/gcc/testsuite/g++.dg/lookup/using24.C trunk/gcc/testsuite/g++.dg/lookup/using25.C trunk/gcc/testsuite/g++.dg/lookup/using26.C trunk/gcc/testsuite/g++.dg/lookup/using27.C trunk/gcc/testsuite/g++.dg/lookup/using28.C trunk/gcc/testsuite/g++.dg/lookup/using29.C trunk/gcc/testsuite/g++.dg/lookup/using30.C trunk/gcc/testsuite/g++.dg/lookup/using31.C trunk/gcc/testsuite/g++.dg/lookup/using32.C trunk/gcc/testsuite/g++.dg/lookup/using33.C trunk/gcc/testsuite/g++.dg/lookup/using34.C trunk/gcc/testsuite/g++.dg/lookup/using35.C trunk/gcc/testsuite/g++.dg/lookup/using36.C trunk/gcc/testsuite/g++.dg/lookup/using37.C trunk/gcc/testsuite/g++.dg/lookup/using38.C trunk/gcc/testsuite/g++.dg/lookup/using39.C trunk/gcc/testsuite/g++.dg/lookup/using40.C trunk/gcc/testsuite/g++.dg/lookup/using41.C trunk/gcc/testsuite/g++.dg/lookup/using42.C trunk/gcc/testsuite/g++.dg/lookup/using44.C trunk/gcc/testsuite/g++.dg/lookup/using45.C Modified: trunk/gcc/cp/ChangeLog trunk/gcc/cp/class.c trunk/gcc/cp/cp-tree.h trunk/gcc/cp/decl.c trunk/gcc/cp/name-lookup.c trunk/gcc/cp/parser.c trunk/gcc/cp/search.c trunk/gcc/cp/semantics.c trunk/gcc/cp/typeck.c trunk/gcc/testsuite/ChangeLog trunk/gcc/testsuite/g++.dg/abi/mangle41.C trunk/gcc/testsuite/g++.dg/inherit/using4.C trunk/gcc/testsuite/g++.dg/lookup/name-clash9.C trunk/gcc/testsuite/g++.dg/parse/ctor5.C trunk/gcc/testsuite/g++.dg/template/static4.C trunk/gcc/testsuite/g++.dg/template/typedef1.C trunk/gcc/testsuite/g++.dg/template/using2.C trunk/gcc/testsuite/g++.old-deja/g++.brendan/misc14.C trunk/gcc/testsuite/g++.old-deja/g++.bugs/900127_02.C trunk/gcc/testsuite/g++.old-deja/g++.jason/scoping16.C trunk/gcc/testsuite/g++.old-deja/g++.other/anon7.C trunk/gcc/testsuite/g++.old-deja/g++.other/redecl1.C trunk/gcc/testsuite/g++.old-deja/g++.other/typedef7.C trunk/gcc/testsuite/g++.old-deja/g++.other/using1.C
next parent reply other threads:[~2011-11-14 17:57 UTC|newest] Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top [not found] <bug-30195-4@http.gcc.gnu.org/bugzilla/> 2011-11-14 17:59 ` jason at gcc dot gnu.org [this message] 2011-11-16 8:31 ` fabien at gcc dot gnu.org 2012-01-03 10:02 ` fabien at gcc dot gnu.org 2006-12-13 6:09 [Bug c++/30195] New: " s__nakayama at infoseek dot jp 2006-12-13 6:55 ` [Bug c++/30195] " pinskia at gcc dot gnu dot org 2006-12-30 18:42 ` gdr at gcc dot gnu dot org 2009-01-26 19:14 ` pinskia at gcc dot gnu dot org 2010-05-09 22:09 ` fabien dot chene at gmail dot com 2010-06-07 13:17 ` fabien at gcc dot gnu dot org
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=bug-30195-4-SM6T6SOCUM@http.gcc.gnu.org/bugzilla/ \ --to=gcc-bugzilla@gcc.gnu.org \ --cc=gcc-bugs@gcc.gnu.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).