On 12 August 2014 11:47, Ville Voutilainen ville.voutilai...@gmail.com wrote:
We shouldn't allow friend declarations and member templates to
have virt-specifiers, as neither of those can be virtual. Tested on
Linux x86-64.
Note that the standardese can be squinted at in a way that would allow
virt-specifiers on friends, there's a DR coming for that. I do not expect
the resolution of that DR to be anything but banning virt-specifiers
on such declarations. Clang has very recently made a similar fix for
friend declarations, and it already rejected virt-specifiers on member
templates.
Minor tweak to the diagnostic for friends, say friend declaration instead
of member declared as a friend. Changelog unchanged, attached for
convenience.
diff --git a/gcc/cp/friend.c b/gcc/cp/friend.c
index a30918c..9b37d86 100644
--- a/gcc/cp/friend.c
+++ b/gcc/cp/friend.c
@@ -426,6 +426,10 @@ do_friend (tree ctype, tree declarator, tree decl,
/* Every decl that gets here is a friend of something. */
DECL_FRIEND_P (decl) = 1;
+
+ if (DECL_OVERRIDE_P (decl) || DECL_FINAL_P (decl))
+error (friend declaration %qD may not have virt-specifiers,
+ decl);
/* Unfortunately, we have to handle attributes here. Normally we would
handle them in start_decl_1, but since this is a friend decl start_decl_1
diff --git a/gcc/cp/pt.c b/gcc/cp/pt.c
index 2592172..90b2720 100644
--- a/gcc/cp/pt.c
+++ b/gcc/cp/pt.c
@@ -4737,6 +4737,11 @@ push_template_decl_real (tree decl, bool is_friend)
}
else if (TREE_CODE (decl) == FUNCTION_DECL)
{
+ if (member_template_p)
+ {
+ if (DECL_OVERRIDE_P (decl) || DECL_FINAL_P (decl))
+ error (member template %qD may not have virt-specifiers,
decl);
+ }
if (DECL_DESTRUCTOR_P (decl))
{
/* [temp.mem]
diff --git a/gcc/testsuite/g++.dg/cpp0x/override1.C
b/gcc/testsuite/g++.dg/cpp0x/override1.C
index f5f00ee..05d7290 100644
--- a/gcc/testsuite/g++.dg/cpp0x/override1.C
+++ b/gcc/testsuite/g++.dg/cpp0x/override1.C
@@ -52,6 +52,14 @@ struct D5 : B
void D5::g() override {} // { dg-error not allowed outside a class
definition }
void g() override {} // { dg-error not allowed outside a class definition }
+struct B5
+{
+ friend void f() final; // { dg-error may not have virt-specifiers }
+ friend void g() override; // { dg-error may not have virt-specifiers }
+ template class T void h() final; // { dg-error may not have
virt-specifiers }
+ template class T void i() override; // { dg-error may not have
virt-specifiers }
+};
+
int main()
{
D2B d;
friend-member-template-virt-specifiers.changelog
Description: Binary data