class.c (handle_using_decl): Always use OVL_CURRENT.

	* class.c (handle_using_decl): Always use OVL_CURRENT.
	(resolve_address_of_overloaded_function): Move iterator decl into
	for scope.  Don't strip anticipated decls here.

From-SVN: r247864

gcc/cp/ChangeLog

 2017-05-10  Nathan Sidwell  <nathan@acm.org>
 
+	* class.c (handle_using_decl): Always use OVL_CURRENT.
+	(resolve_address_of_overloaded_function): Move iterator decl into
+	for scope.  Don't strip anticipated decls here.
+
 	* pt.c (print_candidates_1): Separate TREE_LIST and OVERLOAD
 	printing.
 	(print_candidates): Adjust.

gcc/cp/class.c

@@ -1359,7 +1359,6 @@ handle_using_decl (tree using_decl, tree t)
 			     tf_warning_or_error);
   if (old_value)
     {
-      if (is_overloaded_fn (old_value))
       old_value = OVL_CURRENT (old_value);
 
       if (DECL_P (old_value) && DECL_CONTEXT (old_value) == t)
@@ -1384,7 +1383,7 @@ handle_using_decl (tree using_decl, tree t)
 	{
 	  error ("%q+D invalid in %q#T", using_decl, t);
 	  error ("  because of local method %q+#D with same name",
-		 OVL_CURRENT (old_value));
+		 old_value);
 	  return;
 	}
     }
@@ -8184,10 +8183,7 @@ resolve_address_of_overloaded_function (tree target_type,
      if we're just going to throw them out anyhow.  But, of course, we
      can only do this when we don't *need* a template function.  */
   if (!template_only)
-    {
-      tree fns;
-
-      for (fns = overload; fns; fns = OVL_NEXT (fns))
+    for (tree fns = overload; fns; fns = OVL_NEXT (fns))
       {
 	tree fn = OVL_CURRENT (fns);
 
@@ -8195,17 +8191,11 @@ resolve_address_of_overloaded_function (tree target_type,
 	  /* We're not looking for templates just yet.  */
 	  continue;
 
-	  if ((TREE_CODE (TREE_TYPE (fn)) == METHOD_TYPE)
-	      != is_ptrmem)
+	if ((TREE_CODE (TREE_TYPE (fn)) == METHOD_TYPE) != is_ptrmem)
 	  /* We're looking for a non-static member, and this isn't
 	     one, or vice versa.  */
 	  continue;
 
-	  /* Ignore functions which haven't been explicitly
-	     declared.  */
-	  if (DECL_ANTICIPATED (fn))
-	    continue;
-
 	/* In C++17 we need the noexcept-qualifier to compare types.  */
 	if (flag_noexcept_type)
 	  maybe_instantiate_noexcept (fn);
@@ -8216,7 +8206,6 @@ resolve_address_of_overloaded_function (tree target_type,
 	    || fnptr_conv_p (target_fn_type, fntype))
 	  matches = tree_cons (fn, NULL_TREE, matches);
       }
-    }
 
   /* Now, if we've already got a match (or matches), there's no need
      to proceed to the template functions.  But, if we don't have a