From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 71411 invoked by alias); 28 Nov 2018 15:48:52 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 71314 invoked by uid 89); 28 Nov 2018 15:48:48 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-26.9 required=5.0 tests=BAYES_00,GIT_PATCH_0,GIT_PATCH_1,GIT_PATCH_2,GIT_PATCH_3,SPF_HELO_PASS autolearn=ham version=3.3.2 spammy=marek, address_p, sk:finish_, Polacek X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Wed, 28 Nov 2018 15:48:46 +0000 Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 7A92B30014AD for ; Wed, 28 Nov 2018 15:48:43 +0000 (UTC) Received: from redhat.com (ovpn-125-91.rdu2.redhat.com [10.10.125.91]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 04DBC6838D; Wed, 28 Nov 2018 15:48:42 +0000 (UTC) Date: Wed, 28 Nov 2018 15:48:00 -0000 From: Marek Polacek To: GCC Patches , Jason Merrill Subject: C++ PATCH for c++/88184, ICE when treating name as template-name Message-ID: <20181128154849.GD3269@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.10.1 (2018-07-13) X-SW-Source: 2018-11/txt/msg02314.txt.bz2 Since P0846R0 was implemented, a name will be treated as a template-name when it is an unqualified-id followed by a < and name lookup finds either one or more functions or finds nothing, in order to potentially cause ADL to be performed. In this case, we had f (); where we'd found a decl for f (not a TEMPLATE_DECL) and < follows. But there are no arguments, so ADL can't be performed, so I think let's diagnose this problem sooner lest things break later in finish_call_expr. Bootstrapped/regtested on x86_64-linux, ok for trunk? 2018-11-28 Marek Polacek PR c++/88184, ICE when treating name as template-name. * parser.c (cp_parser_postfix_expression): Give error when the name-as-template-name doesn't have any arguments. * g++.dg/cpp2a/fn-template17.C: New test. * g++.dg/cpp2a/fn-template18.C: New test. diff --git gcc/cp/parser.c gcc/cp/parser.c index 3a98ed900cd..01097b57eb9 100644 --- gcc/cp/parser.c +++ gcc/cp/parser.c @@ -7241,6 +7241,20 @@ cp_parser_postfix_expression (cp_parser *parser, bool address_p, bool cast_p, complain); } } + else if (TREE_CODE (postfix_expression) == TEMPLATE_ID_EXPR + && (TREE_CODE (TREE_OPERAND (postfix_expression, 0)) + == FUNCTION_DECL) + && args->is_empty ()) + { + /* We can get here when treating a name as a template name + (because it was an unqualified-id followed by a < and + name lookup found a decl), but now we see that there + are no arguments, so ADL isn't possible. */ + auto_diagnostic_group d; + error_at (loc, "no matching function for call to %<%D()%>", + postfix_expression.get_value ()); + postfix_expression = error_mark_node; + } } if (TREE_CODE (postfix_expression) == COMPONENT_REF) diff --git gcc/testsuite/g++.dg/cpp2a/fn-template17.C gcc/testsuite/g++.dg/cpp2a/fn-template17.C new file mode 100644 index 00000000000..f0d24107682 --- /dev/null +++ gcc/testsuite/g++.dg/cpp2a/fn-template17.C @@ -0,0 +1,21 @@ +// PR c++/88184 +// { dg-do compile } +// { dg-options "-std=c++2a -fchecking=2" } + +namespace A +{ + void f (); +} + +using A::f; + +template void g () +{ + f (); // { dg-error "no matching function for call" } +} + +void +fn () +{ + g(); +} diff --git gcc/testsuite/g++.dg/cpp2a/fn-template18.C gcc/testsuite/g++.dg/cpp2a/fn-template18.C new file mode 100644 index 00000000000..7fe5e89ace3 --- /dev/null +++ gcc/testsuite/g++.dg/cpp2a/fn-template18.C @@ -0,0 +1,23 @@ +// PR c++/88184 +// { dg-do compile } +// { dg-options "-std=c++2a -fchecking=2" } + +namespace A +{ + void f (); + void f (int); + void f (int, int); +} + +using A::f; + +template void g () +{ + f (); // { dg-error "no matching function for call" } +} + +void +fn () +{ + g(); +}