diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog index 7b40665484f..93e639e8687 100644 --- a/gcc/cp/ChangeLog +++ b/gcc/cp/ChangeLog @@ -1,3 +1,9 @@ +2013-06-27 Marc Glisse + + PR c++/57172 + * pt.c (more_specialized_fn): If both arguments are references, + give priority to an lvalue. + 2013-06-26 Jason Merrill * typeck2.c (store_init_value): Diagnose a non-constant diff --git a/gcc/cp/pt.c b/gcc/cp/pt.c index 667e37f2ec9..e2ffe7314c4 100644 --- a/gcc/cp/pt.c +++ b/gcc/cp/pt.c @@ -17510,7 +17510,8 @@ check_undeduced_parms (tree targs, tree args, tree end) corresponding argument. Deduction is done as for class templates. The arguments used in deduction have reference and top level cv qualifiers removed. Iff both arguments were originally reference - types *and* deduction succeeds in both directions, the template + types *and* deduction succeeds in both directions, an lvalue reference + wins against an rvalue reference and otherwise the template with the more cv-qualified argument wins for that pairing (if neither is more cv-qualified, they both are equal). Unlike regular deduction, after all the arguments have been deduced in this way, @@ -17586,6 +17587,8 @@ more_specialized_fn (tree pat1, tree pat2, int len) int deduce1, deduce2; int quals1 = -1; int quals2 = -1; + int ref1 = 0; + int ref2 = 0; if (TREE_CODE (arg1) == TYPE_PACK_EXPANSION && TREE_CODE (arg2) == TYPE_PACK_EXPANSION) @@ -17601,12 +17604,14 @@ more_specialized_fn (tree pat1, tree pat2, int len) if (TREE_CODE (arg1) == REFERENCE_TYPE) { + ref1 = TYPE_REF_IS_RVALUE (arg1) + 1; arg1 = TREE_TYPE (arg1); quals1 = cp_type_quals (arg1); } if (TREE_CODE (arg2) == REFERENCE_TYPE) { + ref2 = TYPE_REF_IS_RVALUE (arg2) + 1; arg2 = TREE_TYPE (arg2); quals2 = cp_type_quals (arg2); } @@ -17684,19 +17689,33 @@ more_specialized_fn (tree pat1, tree pat2, int len) /* "If, for a given type, deduction succeeds in both directions (i.e., the types are identical after the transformations above) - and if the type from the argument template is more cv-qualified - than the type from the parameter template (as described above) - that type is considered to be more specialized than the other. If - neither type is more cv-qualified than the other then neither type - is more specialized than the other." */ + and both P and A were reference types (before being replaced with + the type referred to above): + - if the type from the argument template was an lvalue reference and + the type from the parameter template was not, the argument type is + considered to be more specialized than the other; otherwise, + - if the type from the argument template is more cv-qualified + than the type from the parameter template (as described above), + the argument type is considered to be more specialized than the other; + otherwise, + - neither type is more specialized than the other." */ - if (deduce1 && deduce2 - && quals1 != quals2 && quals1 >= 0 && quals2 >= 0) + if (deduce1 && deduce2) { - if ((quals1 & quals2) == quals2) - lose2 = true; - if ((quals1 & quals2) == quals1) - lose1 = true; + if (ref1 && ref2 && ref1 != ref2) + { + if (ref1 > ref2) + lose1 = true; + else + lose2 = true; + } + else if (quals1 != quals2 && quals1 >= 0 && quals2 >= 0) + { + if ((quals1 & quals2) == quals2) + lose2 = true; + if ((quals1 & quals2) == quals1) + lose1 = true; + } } if (lose1 && lose2) diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index f6ab2ba0468..73cf781d1fe 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2013-06-27 Marc Glisse + + PR c++/57172 + * g++.dg/cpp0x/pr57172.C: New testcase. + 2013-06-27 Andreas Krebbel * gcc.target/s390/htm-1.c: New file. diff --git a/gcc/testsuite/g++.dg/cpp0x/pr57172.C b/gcc/testsuite/g++.dg/cpp0x/pr57172.C new file mode 100644 index 00000000000..21088381360 --- /dev/null +++ b/gcc/testsuite/g++.dg/cpp0x/pr57172.C @@ -0,0 +1,7 @@ +// PR c++/57172 +// { dg-do compile { target c++11 } } + +template int f (T&) { return 0; } +template int f (T&&) = delete; +int i; +int j = f (i);