re PR c++/48118 (g++ sometimes allows copying a volatile class)

	PR c++/48118
	* call.c (build_over_call): Don't skip ck_rvalue.

From-SVN: r171147

gcc/cp/ChangeLog

+2011-03-18  Jason Merrill  <jason@redhat.com>
+
+	PR c++/48118
+	* call.c (build_over_call): Don't skip ck_rvalue.
+
 2011-03-17  Jason Merrill  <jason@redhat.com>
 
 	PR c++/47504

gcc/cp/call.c

@@ -6168,12 +6168,6 @@ build_over_call (struct z_candidate *cand, int flags, tsubst_flags_t complain)
 
       conv = convs[i];
 
-      /* Don't make a copy here if build_call is going to.  */
-      if (conv->kind == ck_rvalue
-	  && COMPLETE_TYPE_P (complete_type (type))
-	  && !TREE_ADDRESSABLE (type))
-	conv = conv->u.next;
-
       /* Warn about initializer_list deduction that isn't currently in the
 	 working draft.  */
       if (cxx_dialect > cxx98

gcc/testsuite/ChangeLog

 2011-03-18  Jason Merrill  <jason@redhat.com>
 
+	* g++.dg/overload/volatile1.C: New.
+
 	* g++.dg/opt/pr23372.C: New.
 
 	* g++.dg/ext/attrib32.C: Expect errors on the two-names case.

gcc/testsuite/g++.dg/overload/volatile1.C

+// PR c++/48118
+// { dg-prune-output "note" }
+
+struct A { };
+
+void f (A);			// { dg-error "argument 1" }
+void (*g)(A);
+
+int main()
+{
+  volatile A a;
+  f(a);				// { dg-error "no match" }
+  g(a);				// { dg-error "no match" }
+}