]> gcc.gnu.org Git - gcc.git/blobdiff - gcc/cp/call.c
call.c: Fix comment formatting.
[gcc.git] / gcc / cp / call.c
index 00458c0624813633edb9c34080445f838cdb8f94..2dc4f2240c640e0154b5efa43e93efcb3ff653be 100644 (file)
@@ -1,6 +1,6 @@
 /* Functions related to invoking methods and overloaded functions.
-   Copyright (C) 1987, 1992, 1993, 1994, 1995, 1996, 1997, 1998,
-   1999, 2000, 2001, 2002 Free Software Foundation, Inc.
+   Copyright (C) 1987, 1992, 1993, 1994, 1995, 1996, 1997, 1998, 
+   1999, 2000, 2001, 2002, 2003 Free Software Foundation, Inc.
    Contributed by Michael Tiemann (tiemann@cygnus.com) and
    modified by Brendan Kehoe (brendan@cygnus.com).
 
@@ -35,65 +35,66 @@ Boston, MA 02111-1307, USA.  */
 #include "rtl.h"
 #include "toplev.h"
 #include "expr.h"
-#include "ggc.h"
 #include "diagnostic.h"
-
-extern int inhibit_warnings;
+#include "intl.h"
+#include "target.h"
+#include "convert.h"
 
 static tree build_field_call (tree, tree, tree);
 static struct z_candidate * tourney (struct z_candidate *);
 static int equal_functions (tree, tree);
 static int joust (struct z_candidate *, struct z_candidate *, bool);
 static int compare_ics (tree, tree);
-static tree build_over_call (struct z_candidate *, tree, int);
+static tree build_over_call (struct z_candidate *, int);
 static tree build_java_interface_fn_ref (tree, tree);
-#define convert_like(CONV, EXPR) \
-  convert_like_real ((CONV), (EXPR), NULL_TREE, 0, 0)
-#define convert_like_with_context(CONV, EXPR, FN, ARGNO) \
-  convert_like_real ((CONV), (EXPR), (FN), (ARGNO), 0)
-static tree convert_like_real (tree, tree, tree, int, int);
+#define convert_like(CONV, EXPR)                               \
+  convert_like_real ((CONV), (EXPR), NULL_TREE, 0, 0,          \
+                    /*issue_conversion_warnings=*/true)
+#define convert_like_with_context(CONV, EXPR, FN, ARGNO)       \
+  convert_like_real ((CONV), (EXPR), (FN), (ARGNO), 0,                 \
+                    /*issue_conversion_warnings=*/true)
+static tree convert_like_real (tree, tree, tree, int, int, bool);
 static void op_error (enum tree_code, enum tree_code, tree, tree,
                            tree, const char *);
 static tree build_object_call (tree, tree);
 static tree resolve_args (tree);
 static struct z_candidate *build_user_type_conversion_1 (tree, tree, int);
+static void print_z_candidate (const char *, struct z_candidate *);
 static void print_z_candidates (struct z_candidate *);
 static tree build_this (tree);
-static struct z_candidate *splice_viable (struct z_candidate *);
-static bool any_viable (struct z_candidate *);
+static struct z_candidate *splice_viable (struct z_candidate *, bool, bool *);
 static bool any_strictly_viable (struct z_candidate *);
 static struct z_candidate *add_template_candidate
-        (struct z_candidate *, tree, tree, tree, tree, tree, 
+        (struct z_candidate **, tree, tree, tree, tree, tree, 
          tree, tree, int, unification_kind_t);
 static struct z_candidate *add_template_candidate_real
-       (struct z_candidate *, tree, tree, tree, tree, tree, 
+       (struct z_candidate **, tree, tree, tree, tree, tree, 
            tree, tree, int, tree, unification_kind_t);
 static struct z_candidate *add_template_conv_candidate 
-        (struct z_candidate *, tree, tree, tree, tree, tree, tree);
-static struct z_candidate *add_builtin_candidates
-       (struct z_candidate *, enum tree_code, enum tree_code,
+        (struct z_candidate **, tree, tree, tree, tree, tree, tree);
+static void add_builtin_candidates
+       (struct z_candidate **, enum tree_code, enum tree_code,
               tree, tree *, int);
-static struct z_candidate *add_builtin_candidate
-       (struct z_candidate *, enum tree_code, enum tree_code,
+static void add_builtin_candidate
+       (struct z_candidate **, enum tree_code, enum tree_code,
               tree, tree, tree, tree *, tree *, int);
 static bool is_complete (tree);
-static struct z_candidate *build_builtin_candidate 
-       (struct z_candidate *, tree, tree, tree, tree *, tree *,
+static void build_builtin_candidate 
+       (struct z_candidate **, tree, tree, tree, tree *, tree *,
               int);
 static struct z_candidate *add_conv_candidate 
-       (struct z_candidate *, tree, tree, tree, tree, tree);
+       (struct z_candidate **, tree, tree, tree, tree, tree);
 static struct z_candidate *add_function_candidate 
-       (struct z_candidate *, tree, tree, tree, tree, tree, int);
+       (struct z_candidate **, tree, tree, tree, tree, tree, int);
 static tree implicit_conversion (tree, tree, tree, int);
 static tree standard_conversion (tree, tree, tree);
 static tree reference_binding (tree, tree, tree, int);
-static tree non_reference (tree);
 static tree build_conv (enum tree_code, tree, tree);
 static bool is_subseq (tree, tree);
 static tree maybe_handle_ref_bind (tree *);
 static void maybe_handle_implicit_object (tree *);
 static struct z_candidate *add_candidate 
-        (struct z_candidate *, tree, tree, tree, tree, int);
+        (struct z_candidate **, tree, tree, tree, tree, tree, int);
 static tree source_type (tree);
 static void add_warning (struct z_candidate *, struct z_candidate *);
 static bool reference_related_p (tree, tree);
@@ -103,7 +104,12 @@ static tree direct_reference_binding (tree, tree);
 static bool promoted_arithmetic_type_p (tree);
 static tree conditional_conversion (tree, tree);
 static char *name_as_c_string (tree, tree, bool *);
-static tree call_builtin_trap (void);
+static tree call_builtin_trap (tree);
+static tree prep_operand (tree);
+static void add_candidates (tree, tree, tree, bool, tree, tree,
+                           int, struct z_candidate **);
+static tree merge_conversion_sequences (tree, tree);
+static bool magic_varargs_p (tree);
 
 tree
 build_vfield_ref (tree datum, tree type)
@@ -146,8 +152,8 @@ build_field_call (tree instance_ptr, tree decl, tree parms)
        return error_mark_node;
 
       if (IS_AGGR_TYPE (TREE_TYPE (instance)))
-       return build_opfncall (CALL_EXPR, LOOKUP_NORMAL,
-                              instance, parms, NULL_TREE);
+       return build_new_op (CALL_EXPR, LOOKUP_NORMAL,
+                            instance, parms, NULL_TREE);
       else if (TREE_CODE (TREE_TYPE (instance)) == FUNCTION_TYPE
               || (TREE_CODE (TREE_TYPE (instance)) == POINTER_TYPE
                   && (TREE_CODE (TREE_TYPE (TREE_TYPE (instance)))
@@ -201,106 +207,6 @@ check_dtor_name (tree basetype, tree name)
   return false;
 }
 
-/* Build a method call of the form `EXP->SCOPES::NAME (PARMS)'.
-   This is how virtual function calls are avoided.  */
-
-tree
-build_scoped_method_call (tree exp, tree basetype, tree name, tree parms)
-{
-  /* Because this syntactic form does not allow
-     a pointer to a base class to be `stolen',
-     we need not protect the derived->base conversion
-     that happens here.
-     
-     @@ But we do have to check access privileges later.  */
-  tree binfo, decl;
-  tree type = TREE_TYPE (exp);
-
-  if (type == error_mark_node
-      || basetype == error_mark_node)
-    return error_mark_node;
-
-  if (processing_template_decl)
-    {
-      name = build_min_nt (SCOPE_REF, basetype, name);
-      return build_min_nt (METHOD_CALL_EXPR, name, exp, parms, NULL_TREE);
-    }
-
-  if (TREE_CODE (type) == REFERENCE_TYPE)
-    type = TREE_TYPE (type);
-
-  if (TREE_CODE (basetype) == TREE_VEC)
-    {
-      binfo = basetype;
-      basetype = BINFO_TYPE (binfo);
-    }
-  else
-    binfo = NULL_TREE;
-
-  /* Check the destructor call syntax.  */
-  if (TREE_CODE (name) == BIT_NOT_EXPR)
-    {
-      /* We can get here if someone writes their destructor call like
-        `obj.NS::~T()'; this isn't really a scoped method call, so hand
-        it off.  */
-      if (TREE_CODE (basetype) == NAMESPACE_DECL)
-       return build_method_call (exp, name, parms, NULL_TREE, LOOKUP_NORMAL);
-
-      if (! check_dtor_name (basetype, name))
-       error ("qualified type `%T' does not match destructor name `~%T'",
-                 basetype, TREE_OPERAND (name, 0));
-
-      /* Destructors can be "called" for simple types; see 5.2.4 and 12.4 Note
-        that explicit ~int is caught in the parser; this deals with typedefs
-        and template parms.  */
-      if (! IS_AGGR_TYPE (basetype))
-       {
-         if (TYPE_MAIN_VARIANT (type) != TYPE_MAIN_VARIANT (basetype))
-           error ("type of `%E' does not match destructor type `%T' (type was `%T')",
-                     exp, basetype, type);
-
-         return cp_convert (void_type_node, exp);
-       }
-    }
-
-  if (TREE_CODE (basetype) == NAMESPACE_DECL)
-    {
-      error ("`%D' is a namespace", basetype);
-      return error_mark_node;
-    }
-  if (! is_aggr_type (basetype, 1))
-    return error_mark_node;
-
-  if (! IS_AGGR_TYPE (type))
-    {
-      error ("base object `%E' of scoped method call is of non-aggregate type `%T'",
-               exp, type);
-      return error_mark_node;
-    }
-
-  decl = build_scoped_ref (exp, basetype, &binfo);
-
-  if (binfo)
-    {
-      /* Call to a destructor.  */
-      if (TREE_CODE (name) == BIT_NOT_EXPR)
-       {
-         if (! TYPE_HAS_DESTRUCTOR (TREE_TYPE (decl)))
-           return cp_convert (void_type_node, exp);
-         
-         return build_delete (TREE_TYPE (decl), decl, 
-                              sfk_complete_destructor,
-                              LOOKUP_NORMAL|LOOKUP_NONVIRTUAL|LOOKUP_DESTRUCTOR,
-                              0);
-       }
-
-      /* Call to a method.  */
-      return build_method_call (decl, name, parms, binfo,
-                               LOOKUP_NORMAL|LOOKUP_NONVIRTUAL);
-    }
-  return error_mark_node;
-}
-
 /* We want the address of a function or method.  We avoid creating a
    pointer-to-member function.  */
 
@@ -313,24 +219,16 @@ build_addr_func (tree function)
      functions.  */
   if (TREE_CODE (type) == METHOD_TYPE)
     {
-      tree addr;
-
-      type = build_pointer_type (type);
-
-      if (!cxx_mark_addressable (function))
-       return error_mark_node;
-
-      addr = build1 (ADDR_EXPR, type, function);
-
-      /* Address of a static or external variable or function counts
-        as a constant */
-      if (staticp (function))
-       TREE_CONSTANT (addr) = 1;
-
-      function = addr;
+      if (TREE_CODE (function) == OFFSET_REF)
+       {
+         tree object = build_address (TREE_OPERAND (function, 0));
+         return get_member_function_from_ptrfunc (&object,
+                                                  TREE_OPERAND (function, 1));
+       }
+      function = build_address (function);
     }
   else
-    function = default_conversion (function);
+    function = decay_conversion (function);
 
   return function;
 }
@@ -406,10 +304,8 @@ build_call (tree function, tree parms)
                                    TREE_VALUE (tmp), t);
        }
 
-  function = build_nt (CALL_EXPR, function, parms, NULL_TREE);
+  function = build (CALL_EXPR, result_type, function, parms);
   TREE_HAS_CONSTRUCTOR (function) = is_constructor;
-  TREE_TYPE (function) = result_type;
-  TREE_SIDE_EFFECTS (function) = 1;
   TREE_NOTHROW (function) = nothrow;
   
   return function;
@@ -430,8 +326,7 @@ build_call (tree function, tree parms)
 
    BASETYPE_PATH, if non-NULL, contains a chain from the type of INSTANCE
    down to the real instance type to use for access checking.  We need this
-   information to get protected accesses correct.  This parameter is used
-   by build_member_call.
+   information to get protected accesses correct.
 
    FLAGS is the logical disjunction of zero or more LOOKUP_
    flags.  See cp-tree.h for more info.
@@ -464,17 +359,13 @@ build_method_call (tree instance, tree name, tree parms,
   n_build_method_call++;
 #endif
 
-  if (instance == error_mark_node
+  if (error_operand_p (instance)
       || name == error_mark_node
-      || parms == error_mark_node
-      || (instance && TREE_TYPE (instance) == error_mark_node))
+      || parms == error_mark_node)
     return error_mark_node;
 
-  if (processing_template_decl)
-    return build_min_nt (METHOD_CALL_EXPR, name, instance, parms, NULL_TREE);
+  my_friendly_assert (!processing_template_decl, 20030707);
 
-  if (TREE_CODE (instance) == OFFSET_REF)
-    instance = resolve_offset_ref (instance);
   if (TREE_CODE (TREE_TYPE (instance)) == REFERENCE_TYPE)
     instance = convert_from_reference (instance);
   object_type = TREE_TYPE (instance);
@@ -492,7 +383,7 @@ build_method_call (tree instance, tree name, tree parms,
           TREE_OPERAND (name, 0), object_type);
 
       if (! TYPE_HAS_DESTRUCTOR (complete_type (object_type)))
-       return cp_convert (void_type_node, instance);
+       return convert_to_void (instance, /*implicit=*/NULL);
       instance = default_conversion (instance);
       instance_ptr = build_unary_op (ADDR_EXPR, instance, 0);
       return build_delete (build_pointer_type (object_type),
@@ -522,9 +413,9 @@ build_method_call (tree instance, tree name, tree parms,
   if (has_template_args)
     fn = lookup_fnfields (object_type, name, /*protect=*/2);
   else
-    fn = lookup_member (object_type, name, /*protect=*/2, /*want_type=*/0);
+    fn = lookup_member (object_type, name, /*protect=*/2, /*want_type=*/false);
   
-  if (fn && TREE_CODE (fn) == TREE_LIST && !BASELINK_P (fn))
+  if (fn && TREE_CODE (fn) == TREE_LIST)
     {
       error ("request for member `%D' is ambiguous", name);
       print_candidates (fn);
@@ -533,10 +424,7 @@ build_method_call (tree instance, tree name, tree parms,
 
   /* If the name could not be found, issue an error.  */
   if (!fn)
-    {
-      unqualified_name_lookup_error (name);
-      return error_mark_node;
-    }
+    return unqualified_name_lookup_error (name);
 
   if (BASELINK_P (fn) && has_template_args)
     BASELINK_FUNCTIONS (fn)
@@ -556,7 +444,14 @@ struct z_candidate GTY(()) {
   /* The FUNCTION_DECL that will be called if this candidate is
      selected by overload resolution.  */
   tree fn;
+  /* The arguments to use when calling this function.  */
+  tree args;
+  /* The implicit conversion sequences for each of the arguments to
+     FN.  */
   tree convs;
+  /* If FN is a user-defined conversion, the standard conversion
+     sequence from the type returned by FN to the desired destination
+     type.  */
   tree second_conv;
   int viable;
   /* If FN is a member function, the binfo indicating the path used to
@@ -663,28 +558,17 @@ build_conv (enum tree_code code, tree type, tree from)
       break;
     }
   ICS_STD_RANK (t) = rank;
-  ICS_USER_FLAG (t) = ICS_USER_FLAG (from);
+  ICS_USER_FLAG (t) = (code == USER_CONV || ICS_USER_FLAG (from));
   ICS_BAD_FLAG (t) = ICS_BAD_FLAG (from);
   return t;
 }
 
-/* If T is a REFERENCE_TYPE return the type to which T refers.
-   Otherwise, return T itself.  */
-
-static tree
-non_reference (tree t)
-{
-  if (TREE_CODE (t) == REFERENCE_TYPE)
-    t = TREE_TYPE (t);
-  return t;
-}
-
 tree
 strip_top_quals (tree t)
 {
   if (TREE_CODE (t) == ARRAY_TYPE)
     return t;
-  return TYPE_MAIN_VARIANT (t);
+  return cp_build_qualified_type (t, 0);
 }
 
 /* Returns the standard conversion path (see [conv]) from type FROM to type
@@ -698,8 +582,7 @@ standard_conversion (tree to, tree from, tree expr)
   tree conv;
   bool fromref = false;
 
-  if (TREE_CODE (to) == REFERENCE_TYPE)
-    to = TREE_TYPE (to);
+  to = non_reference (to);
   if (TREE_CODE (from) == REFERENCE_TYPE)
     {
       fromref = true;
@@ -711,7 +594,7 @@ standard_conversion (tree to, tree from, tree expr)
   if ((TYPE_PTRFN_P (to) || TYPE_PTRMEMFUNC_P (to))
       && expr && type_unknown_p (expr))
     {
-      expr = instantiate_type (to, expr, tf_none);
+      expr = instantiate_type (to, expr, tf_conv);
       if (expr == error_mark_node)
        return NULL_TREE;
       from = TREE_TYPE (expr);
@@ -737,7 +620,7 @@ standard_conversion (tree to, tree from, tree expr)
   else if (fromref || (expr && lvalue_p (expr)))
     conv = build_conv (RVALUE_CONV, from, conv);
 
-   /* Allow conversion between `__complex__' data types  */
+   /* Allow conversion between `__complex__' data types.  */
   if (tcode == COMPLEX_TYPE && fcode == COMPLEX_TYPE)
     {
       /* The standard conversion sequence to convert FROM to TO is
@@ -760,11 +643,15 @@ standard_conversion (tree to, tree from, tree expr)
   if (same_type_p (from, to))
     return conv;
 
-  if ((tcode == POINTER_TYPE || TYPE_PTRMEMFUNC_P (to))
+  if ((tcode == POINTER_TYPE || TYPE_PTR_TO_MEMBER_P (to))
       && expr && null_ptr_cst_p (expr))
-    {
-      conv = build_conv (STD_CONV, to, conv);
-    }
+    conv = build_conv (STD_CONV, to, conv);
+  else if (tcode == POINTER_TYPE && fcode == POINTER_TYPE
+          && TREE_CODE (TREE_TYPE (to)) == VECTOR_TYPE
+          && TREE_CODE (TREE_TYPE (from)) == VECTOR_TYPE
+          && ((*targetm.vector_opaque_p) (TREE_TYPE (to))
+              || (*targetm.vector_opaque_p) (TREE_TYPE (from))))
+    conv = build_conv (STD_CONV, to, conv);
   else if ((tcode == INTEGER_TYPE && fcode == POINTER_TYPE)
           || (tcode == POINTER_TYPE && fcode == INTEGER_TYPE))
     {
@@ -773,65 +660,88 @@ standard_conversion (tree to, tree from, tree expr)
       conv = build_conv (STD_CONV, to, conv);
       ICS_BAD_FLAG (conv) = 1;
     }
-  else if (tcode == ENUMERAL_TYPE && fcode == INTEGER_TYPE
-          && TYPE_PRECISION (to) == TYPE_PRECISION (from))
+  else if (tcode == ENUMERAL_TYPE && fcode == INTEGER_TYPE)
     {
       /* For backwards brain damage compatibility, allow interconversion of
         enums and integers with a pedwarn.  */
       conv = build_conv (STD_CONV, to, conv);
       ICS_BAD_FLAG (conv) = 1;
     }
-  else if (tcode == POINTER_TYPE && fcode == POINTER_TYPE)
+  else if ((tcode == POINTER_TYPE && fcode == POINTER_TYPE)
+          || (TYPE_PTRMEM_P (to) && TYPE_PTRMEM_P (from)))
     {
-      enum tree_code ufcode = TREE_CODE (TREE_TYPE (from));
-      enum tree_code utcode = TREE_CODE (TREE_TYPE (to));
+      tree to_pointee;
+      tree from_pointee;
 
-      if (same_type_ignoring_top_level_qualifiers_p (TREE_TYPE (from),
-                                                    TREE_TYPE (to)))
+      if (tcode == POINTER_TYPE
+         && same_type_ignoring_top_level_qualifiers_p (TREE_TYPE (from),
+                                                       TREE_TYPE (to)))
        ;
-      else if (utcode == VOID_TYPE && ufcode != OFFSET_TYPE
-              && ufcode != FUNCTION_TYPE)
+      else if (VOID_TYPE_P (TREE_TYPE (to))
+              && !TYPE_PTRMEM_P (from)
+              && TREE_CODE (TREE_TYPE (from)) != FUNCTION_TYPE)
        {
          from = build_pointer_type
            (cp_build_qualified_type (void_type_node, 
                                      cp_type_quals (TREE_TYPE (from))));
          conv = build_conv (PTR_CONV, from, conv);
        }
-      else if (ufcode == OFFSET_TYPE && utcode == OFFSET_TYPE)
+      else if (TYPE_PTRMEM_P (from))
        {
-         tree fbase = TYPE_OFFSET_BASETYPE (TREE_TYPE (from));
-         tree tbase = TYPE_OFFSET_BASETYPE (TREE_TYPE (to));
+         tree fbase = TYPE_PTRMEM_CLASS_TYPE (from);
+         tree tbase = TYPE_PTRMEM_CLASS_TYPE (to);
 
          if (DERIVED_FROM_P (fbase, tbase)
              && (same_type_ignoring_top_level_qualifiers_p
-                 (TREE_TYPE (TREE_TYPE (from)),
-                  TREE_TYPE (TREE_TYPE (to)))))
+                 (TYPE_PTRMEM_POINTED_TO_TYPE (from),
+                  TYPE_PTRMEM_POINTED_TO_TYPE (to))))
            {
-             from = build_ptrmem_type (tbase, TREE_TYPE (TREE_TYPE (from)));
+             from = build_ptrmem_type (tbase, 
+                                       TYPE_PTRMEM_POINTED_TO_TYPE (from));
              conv = build_conv (PMEM_CONV, from, conv);
            }
        }
       else if (IS_AGGR_TYPE (TREE_TYPE (from))
-              && IS_AGGR_TYPE (TREE_TYPE (to)))
+              && IS_AGGR_TYPE (TREE_TYPE (to))
+              /* [conv.ptr]
+                 
+                 An rvalue of type "pointer to cv D," where D is a
+                 class type, can be converted to an rvalue of type
+                 "pointer to cv B," where B is a base class (clause
+                 _class.derived_) of D.  If B is an inaccessible
+                 (clause _class.access_) or ambiguous
+                 (_class.member.lookup_) base class of D, a program
+                 that necessitates this conversion is ill-formed.  */
+              /* Therefore, we use DERIVED_FROM_P, and not
+                 ACESSIBLY_UNIQUELY_DERIVED_FROM_P, in this test.  */
+              && DERIVED_FROM_P (TREE_TYPE (to), TREE_TYPE (from)))
        {
-         if (DERIVED_FROM_P (TREE_TYPE (to), TREE_TYPE (from)))
-           {
-             from = 
-               cp_build_qualified_type (TREE_TYPE (to),
-                                        cp_type_quals (TREE_TYPE (from)));
-             from = build_pointer_type (from);
-             conv = build_conv (PTR_CONV, from, conv);
-           }
+         from = 
+           cp_build_qualified_type (TREE_TYPE (to),
+                                    cp_type_quals (TREE_TYPE (from)));
+         from = build_pointer_type (from);
+         conv = build_conv (PTR_CONV, from, conv);
+       }
+
+      if (tcode == POINTER_TYPE)
+       {
+         to_pointee = TREE_TYPE (to);
+         from_pointee = TREE_TYPE (from);
+       }
+      else
+       {
+         to_pointee = TYPE_PTRMEM_POINTED_TO_TYPE (to);
+         from_pointee = TYPE_PTRMEM_POINTED_TO_TYPE (from);
        }
 
       if (same_type_p (from, to))
        /* OK */;
-      else if (comp_ptr_ttypes (TREE_TYPE (to), TREE_TYPE (from)))
+      else if (comp_ptr_ttypes (to_pointee, from_pointee))
        conv = build_conv (QUAL_CONV, to, conv);
       else if (expr && string_conv_p (to, expr, 0))
        /* converting from string constant to char *.  */
        conv = build_conv (QUAL_CONV, to, conv);
-      else if (ptr_reasonably_similar (TREE_TYPE (to), TREE_TYPE (from)))
+      else if (ptr_reasonably_similar (to_pointee, from_pointee))
        {
          conv = build_conv (PTR_CONV, to, conv);
          ICS_BAD_FLAG (conv) = 1;
@@ -856,21 +766,33 @@ standard_conversion (tree to, tree from, tree expr)
        return 0;
 
       from = cp_build_qualified_type (tbase, cp_type_quals (fbase));
-      from = build_cplus_method_type (from, TREE_TYPE (fromfn),
-                                     TREE_CHAIN (TYPE_ARG_TYPES (fromfn)));
+      from = build_method_type_directly (from, 
+                                        TREE_TYPE (fromfn),
+                                        TREE_CHAIN (TYPE_ARG_TYPES (fromfn)));
       from = build_ptrmemfunc_type (build_pointer_type (from));
       conv = build_conv (PMEM_CONV, from, conv);
     }
   else if (tcode == BOOLEAN_TYPE)
     {
-      if (! (INTEGRAL_CODE_P (fcode) || fcode == REAL_TYPE
-            || fcode == POINTER_TYPE || TYPE_PTRMEMFUNC_P (from)))
-       return 0;
+      /* [conv.bool]
 
-      conv = build_conv (STD_CONV, to, conv);
-      if (fcode == POINTER_TYPE
-         || (TYPE_PTRMEMFUNC_P (from) && ICS_STD_RANK (conv) < PBOOL_RANK))
-       ICS_STD_RANK (conv) = PBOOL_RANK;
+          An rvalue of arithmetic, enumeration, pointer, or pointer to
+         member type can be converted to an rvalue of type bool.  */
+      if (ARITHMETIC_TYPE_P (from)
+         || fcode == ENUMERAL_TYPE
+         || fcode == POINTER_TYPE
+         || TYPE_PTR_TO_MEMBER_P (from))
+       {
+         conv = build_conv (STD_CONV, to, conv);
+         if (fcode == POINTER_TYPE
+             || TYPE_PTRMEM_P (from)
+             || (TYPE_PTRMEMFUNC_P (from) 
+                 && ICS_STD_RANK (conv) < PBOOL_RANK))
+           ICS_STD_RANK (conv) = PBOOL_RANK;
+         return conv;
+       }
+      
+      return NULL_TREE;
     }
   /* We don't check for ENUMERAL_TYPE here because there are no standard
      conversions to enum type.  */
@@ -882,10 +804,14 @@ standard_conversion (tree to, tree from, tree expr)
       conv = build_conv (STD_CONV, to, conv);
 
       /* Give this a better rank if it's a promotion.  */
-      if (to == type_promotes_to (from)
+      if (same_type_p (to, type_promotes_to (from))
          && ICS_STD_RANK (TREE_OPERAND (conv, 0)) <= PROMO_RANK)
        ICS_STD_RANK (conv) = PROMO_RANK;
     }
+  else if (fcode == VECTOR_TYPE && tcode == VECTOR_TYPE
+      && ((*targetm.vector_opaque_p) (from)
+         || (*targetm.vector_opaque_p) (to)))
+    return build_conv (STD_CONV, to, conv);
   else if (IS_AGGR_TYPE (to) && IS_AGGR_TYPE (from)
           && is_properly_derived_from (from, to))
     {
@@ -945,8 +871,14 @@ convert_class_to_reference (tree t, tree s, tree expr)
   tree conversions;
   tree arglist;
   tree conv;
+  tree reference_type;
   struct z_candidate *candidates;
   struct z_candidate *cand;
+  bool any_viable_p;
+
+  conversions = lookup_conversions (s);
+  if (!conversions)
+    return NULL_TREE;
 
   /* [over.match.ref]
 
@@ -973,10 +905,10 @@ convert_class_to_reference (tree t, tree s, tree expr)
   arglist = build_int_2 (0, 0);
   TREE_TYPE (arglist) = build_pointer_type (s);
   arglist = build_tree_list (NULL_TREE, arglist);
-  
-  for (conversions = lookup_conversions (s);
-       conversions;
-       conversions = TREE_CHAIN (conversions))
+
+  reference_type = build_reference_type (t);
+
+  while (conversions)
     {
       tree fns = TREE_VALUE (conversions);
 
@@ -984,66 +916,91 @@ convert_class_to_reference (tree t, tree s, tree expr)
        {
          tree f = OVL_CURRENT (fns);
          tree t2 = TREE_TYPE (TREE_TYPE (f));
-         struct z_candidate *old_candidates = candidates;
+         
+         cand = NULL;
 
          /* If this is a template function, try to get an exact
              match.  */
          if (TREE_CODE (f) == TEMPLATE_DECL)
            {
-             candidates 
-               = add_template_candidate (candidates,
-                                         f, s,
-                                         NULL_TREE,
-                                         arglist,
-                                         build_reference_type (t),
-                                         TYPE_BINFO (s),
-                                         TREE_PURPOSE (conversions),
-                                         LOOKUP_NORMAL,
-                                         DEDUCE_CONV);
+             cand = add_template_candidate (&candidates,
+                                            f, s,
+                                            NULL_TREE,
+                                            arglist,
+                                            reference_type,
+                                            TYPE_BINFO (s),
+                                            TREE_PURPOSE (conversions),
+                                            LOOKUP_NORMAL,
+                                            DEDUCE_CONV);
              
-             if (candidates != old_candidates)
+             if (cand)
                {
                  /* Now, see if the conversion function really returns
                     an lvalue of the appropriate type.  From the
                     point of view of unification, simply returning an
                     rvalue of the right type is good enough.  */
-                 f = candidates->fn;
+                 f = cand->fn;
                  t2 = TREE_TYPE (TREE_TYPE (f));
                  if (TREE_CODE (t2) != REFERENCE_TYPE
                      || !reference_compatible_p (t, TREE_TYPE (t2)))
-                   candidates = candidates->next;
+                   {
+                     candidates = candidates->next;
+                     cand = NULL;
+                   }
                }
            }
          else if (TREE_CODE (t2) == REFERENCE_TYPE
                   && reference_compatible_p (t, TREE_TYPE (t2)))
-           candidates 
-             = add_function_candidate (candidates, f, s, arglist, 
-                                       TYPE_BINFO (s),         
-                                       TREE_PURPOSE (conversions),
-                                       LOOKUP_NORMAL);
+           cand = add_function_candidate (&candidates, f, s, arglist, 
+                                          TYPE_BINFO (s),      
+                                          TREE_PURPOSE (conversions),
+                                          LOOKUP_NORMAL);
+         
+         if (cand)
+           /* Build a standard conversion sequence indicating the
+              binding from the reference type returned by the
+              function to the desired REFERENCE_TYPE.  */
+           cand->second_conv
+             = (direct_reference_binding 
+                (reference_type, 
+                 build1 (IDENTITY_CONV, 
+                         TREE_TYPE (TREE_TYPE (TREE_TYPE (cand->fn))),
+                         NULL_TREE)));
        }
+      conversions = TREE_CHAIN (conversions);
     }
 
+  candidates = splice_viable (candidates, pedantic, &any_viable_p);
   /* If none of the conversion functions worked out, let our caller
      know.  */
-  if (!any_viable (candidates))
+  if (!any_viable_p)
     return NULL_TREE;
-  
-  candidates = splice_viable (candidates);
+
   cand = tourney (candidates);
   if (!cand)
     return NULL_TREE;
 
-  conv = build1 (IDENTITY_CONV, s, expr);
-  conv = build_conv (USER_CONV, TREE_TYPE (TREE_TYPE (cand->fn)),
-                    conv);
+  /* Now that we know that this is the function we're going to use fix
+     the dummy first argument.  */
+  cand->args = tree_cons (NULL_TREE,
+                         build_this (expr),
+                         TREE_CHAIN (cand->args));
+
+  /* Build a user-defined conversion sequence representing the
+     conversion.  */
+  conv = build_conv (USER_CONV,
+                    TREE_TYPE (TREE_TYPE (cand->fn)),
+                    build1 (IDENTITY_CONV, TREE_TYPE (expr), expr));
   TREE_OPERAND (conv, 1) = build_zc_wrapper (cand);
-  ICS_USER_FLAG (conv) = 1;
+
+  /* Merge it with the standard conversion sequence from the
+     conversion function's return type to the desired type.  */
+  cand->second_conv = merge_conversion_sequences (conv, cand->second_conv);
+
   if (cand->viable == -1)
     ICS_BAD_FLAG (conv) = 1;
-  cand->second_conv = conv;
-
-  return conv;
+  
+  return cand->second_conv;
 }
 
 /* A reference of the indicated TYPE is being bound directly to the
@@ -1053,7 +1010,13 @@ convert_class_to_reference (tree t, tree s, tree expr)
 static tree
 direct_reference_binding (tree type, tree conv)
 {
-  tree t = TREE_TYPE (type);
+  tree t;
+
+  my_friendly_assert (TREE_CODE (type) == REFERENCE_TYPE, 20030306);
+  my_friendly_assert (TREE_CODE (TREE_TYPE (conv)) != REFERENCE_TYPE,
+                     20030306);
+
+  t = TREE_TYPE (type);
 
   /* [over.ics.rank] 
      
@@ -1131,14 +1094,14 @@ reference_binding (tree rto, tree rfrom, tree expr, int flags)
         -- is an lvalue (but not an lvalue for a bit-field), and "cv1 T1"
            is reference-compatible with "cv2 T2,"
         
-        the reference is bound directly to the initializer exprssion
+        the reference is bound directly to the initializer expression
         lvalue.  */
       conv = build1 (IDENTITY_CONV, from, expr);
       conv = direct_reference_binding (rto, conv);
-      if ((lvalue_p & clk_bitfield) != 0 
-         && CP_TYPE_CONST_NON_VOLATILE_P (to))
+      if ((lvalue_p & clk_bitfield) != 0
+         || ((lvalue_p & clk_packed) != 0 && !TYPE_PACKED (to)))
        /* For the purposes of overload resolution, we ignore the fact
-          this expression is a bitfield. (In particular,
+          this expression is a bitfield or packed field. (In particular,
           [over.ics.ref] says specifically that a function with a
           non-const reference parameter is viable even if the
           argument is a bitfield.)
@@ -1149,13 +1112,14 @@ reference_binding (tree rto, tree rfrom, tree expr, int flags)
           a temporary, so we just issue an error when the conversion
           actually occurs.  */
        NEED_TEMPORARY_P (conv) = 1;
+                                       
       return conv;
     }
   else if (CLASS_TYPE_P (from) && !(flags & LOOKUP_NO_CONVERSION))
     {
       /* [dcl.init.ref]
 
-        If the initializer exprsesion
+        If the initializer expression
 
         -- has a class type (i.e., T2 is a class type) can be
            implicitly converted to an lvalue of type "cv3 T3," where
@@ -1168,7 +1132,7 @@ reference_binding (tree rto, tree rfrom, tree expr, int flags)
        in the second case.  */
       conv = convert_class_to_reference (to, from, expr);
       if (conv)
-       return direct_reference_binding (rto, conv);
+       return conv;
     }
 
   /* From this point on, we conceptually need temporaries, even if we
@@ -1202,11 +1166,13 @@ reference_binding (tree rto, tree rfrom, tree expr, int flags)
      -- The reference is bound to the object represented by the rvalue
         or to a sub-object within that object.  
 
-     In this case, the implicit conversion sequence is supposed to be
-     same as we would obtain by generating a temporary.  Fortunately,
-     if the types are reference compatible, then this is either an
-     identity conversion or the derived-to-base conversion, just as
-     for direct binding.  */
+     -- ...
+       
+     We use the first alternative.  The implicit conversion sequence
+     is supposed to be same as we would obtain by generating a
+     temporary.  Fortunately, if the types are reference compatible,
+     then this is either an identity conversion or the derived-to-base
+     conversion, just as for direct binding.  */
   if (CLASS_TYPE_P (from) && compatible_p)
     {
       conv = build1 (IDENTITY_CONV, from, expr);
@@ -1244,37 +1210,26 @@ static tree
 implicit_conversion (tree to, tree from, tree expr, int flags)
 {
   tree conv;
-  struct z_candidate *cand;
-
-  /* Resolve expressions like `A::p' that we thought might become
-     pointers-to-members.  */
-  if (expr && TREE_CODE (expr) == OFFSET_REF)
-    {
-      expr = resolve_offset_ref (expr);
-      from = TREE_TYPE (expr);
-    }
 
   if (from == error_mark_node || to == error_mark_node
       || expr == error_mark_node)
     return NULL_TREE;
 
-  /* Make sure both the FROM and TO types are complete so that
-     user-defined conversions are available.  */
-  complete_type (from);
-  complete_type (to);
-
   if (TREE_CODE (to) == REFERENCE_TYPE)
     conv = reference_binding (to, from, expr, flags);
   else
     conv = standard_conversion (to, from, expr);
 
   if (conv)
-    ;
-  else if (expr != NULL_TREE
-          && (IS_AGGR_TYPE (from)
-              || IS_AGGR_TYPE (to))
-          && (flags & LOOKUP_NO_CONVERSION) == 0)
+    return conv;
+
+  if (expr != NULL_TREE
+      && (IS_AGGR_TYPE (from)
+         || IS_AGGR_TYPE (to))
+      && (flags & LOOKUP_NO_CONVERSION) == 0)
     {
+      struct z_candidate *cand;
+
       cand = build_user_type_conversion_1
        (to, expr, LOOKUP_ONLYCONVERTING);
       if (cand)
@@ -1283,28 +1238,30 @@ implicit_conversion (tree to, tree from, tree expr, int flags)
       /* We used to try to bind a reference to a temporary here, but that
         is now handled by the recursive call to this function at the end
         of reference_binding.  */
+      return conv;
     }
 
-  return conv;
+  return NULL_TREE;
 }
 
 /* Add a new entry to the list of candidates.  Used by the add_*_candidate
    functions.  */
 
 static struct z_candidate *
-add_candidate (struct z_candidate *candidates, 
-              tree fn, tree convs, tree access_path, tree
-              conversion_path, int viable)
+add_candidate (struct z_candidate **candidates, 
+              tree fn, tree args, tree convs, tree access_path, 
+              tree conversion_path, int viable)
 {
-  struct z_candidate *cand
-    = (struct z_candidate *) ggc_alloc_cleared (sizeof (struct z_candidate));
+  struct z_candidate *cand = ggc_alloc_cleared (sizeof (struct z_candidate));
 
   cand->fn = fn;
+  cand->args = args;
   cand->convs = convs;
   cand->access_path = access_path;
   cand->conversion_path = conversion_path;
   cand->viable = viable;
-  cand->next = candidates;
+  cand->next = *candidates;
+  *candidates = cand;
 
   return cand;
 }
@@ -1317,7 +1274,7 @@ add_candidate (struct z_candidate *candidates,
    comes from for purposes of overload resolution.  */
 
 static struct z_candidate *
-add_function_candidate (struct z_candidate *candidates, 
+add_function_candidate (struct z_candidate **candidates, 
                        tree fn, tree ctype, tree arglist, 
                        tree access_path, tree conversion_path,
                        int flags)
@@ -1326,15 +1283,24 @@ add_function_candidate (struct z_candidate *candidates,
   int i, len;
   tree convs;
   tree parmnode, argnode;
+  tree orig_arglist;
   int viable = 1;
 
+  /* Built-in functions that haven't been declared don't really
+     exist.  */
+  if (DECL_ANTICIPATED (fn))
+    return NULL;
+
   /* The `this', `in_chrg' and VTT arguments to constructors are not
      considered in overload resolution.  */
   if (DECL_CONSTRUCTOR_P (fn))
     {
       parmlist = skip_artificial_parms_for (fn, parmlist);
+      orig_arglist = arglist;
       arglist = skip_artificial_parms_for (fn, arglist);
     }
+  else 
+    orig_arglist = arglist;
 
   len = list_length (arglist);
   convs = make_tree_vec (len);
@@ -1432,7 +1398,7 @@ add_function_candidate (struct z_candidate *candidates,
     }
 
  out:
-  return add_candidate (candidates, fn, convs, access_path,
+  return add_candidate (candidates, fn, orig_arglist, convs, access_path,
                        conversion_path, viable);
 }
 
@@ -1448,7 +1414,7 @@ add_function_candidate (struct z_candidate *candidates,
    instead of the function.  */
 
 static struct z_candidate *
-add_conv_candidate (struct z_candidate *candidates, tree fn, tree obj,
+add_conv_candidate (struct z_candidate **candidates, tree fn, tree obj,
                     tree arglist, tree access_path, tree conversion_path)
 {
   tree totype = TREE_TYPE (TREE_TYPE (fn));
@@ -1467,8 +1433,8 @@ add_conv_candidate (struct z_candidate *candidates, tree fn, tree obj,
   flags = LOOKUP_NORMAL;
 
   /* Don't bother looking up the same type twice.  */
-  if (candidates && candidates->fn == totype)
-    return candidates;
+  if (*candidates && (*candidates)->fn == totype)
+    return NULL;
 
   for (i = 0; i < len; ++i)
     {
@@ -1509,12 +1475,12 @@ add_conv_candidate (struct z_candidate *candidates, tree fn, tree obj,
   if (!sufficient_parms_p (parmnode))
     viable = 0;
 
-  return add_candidate (candidates, totype, convs, access_path,
+  return add_candidate (candidates, totype, arglist, convs, access_path,
                        conversion_path, viable);
 }
 
-static struct z_candidate *
-build_builtin_candidate (struct z_candidate *candidates, tree fnname,
+static void
+build_builtin_candidate (struct z_candidate **candidates, tree fnname,
                          tree type1, tree type2, tree *args, tree *argtypes,
                          int flags)
 {
@@ -1556,10 +1522,10 @@ build_builtin_candidate (struct z_candidate *candidates, tree fnname,
        viable = 0;
     }      
 
-  return add_candidate (candidates, fnname, convs, 
-                       /*access_path=*/NULL_TREE,
-                       /*conversion_path=*/NULL_TREE,
-                       viable);
+  add_candidate (candidates, fnname, /*args=*/NULL_TREE, convs, 
+                /*access_path=*/NULL_TREE,
+                /*conversion_path=*/NULL_TREE,
+                viable);
 }
 
 static bool
@@ -1595,8 +1561,8 @@ promoted_arithmetic_type_p (tree type)
    of which TYPE1 and TYPE2 are, we add both candidates
    CODE (TYPE1, TYPE1) and CODE (TYPE2, TYPE2).  */
 
-static struct z_candidate *
-add_builtin_candidate (struct z_candidate *candidates, enum tree_code code,
+static void
+add_builtin_candidate (struct z_candidate **candidates, enum tree_code code,
                        enum tree_code code2, tree fnname, tree type1,
                        tree type2, tree *args, tree *argtypes, int flags)
 {
@@ -1635,7 +1601,7 @@ add_builtin_candidate (struct z_candidate *candidates, enum tree_code code,
     case POSTDECREMENT_EXPR:
     case PREDECREMENT_EXPR:
       if (TREE_CODE (type1) == BOOLEAN_TYPE)
-       return candidates;
+       return;
     case POSTINCREMENT_EXPR:
     case PREINCREMENT_EXPR:
       if (ARITHMETIC_TYPE_P (type1) || TYPE_PTROB_P (type1))
@@ -1643,7 +1609,7 @@ add_builtin_candidate (struct z_candidate *candidates, enum tree_code code,
          type1 = build_reference_type (type1);
          break;
        }
-      return candidates;
+      return;
 
 /* 7 For every cv-qualified or cv-unqualified complete object type T, there
      exist candidate operator functions of the form
@@ -1659,7 +1625,7 @@ add_builtin_candidate (struct z_candidate *candidates, enum tree_code code,
          && (TYPE_PTROB_P (type1)
              || TREE_CODE (TREE_TYPE (type1)) == FUNCTION_TYPE))
        break;
-      return candidates;
+      return
 
 /* 9 For every type T, there exist candidate operator functions of the form
             T*      operator+(T*);
@@ -1670,13 +1636,12 @@ add_builtin_candidate (struct z_candidate *candidates, enum tree_code code,
             T       operator-(T);  */
 
     case CONVERT_EXPR: /* unary + */
-      if (TREE_CODE (type1) == POINTER_TYPE
-         && TREE_CODE (TREE_TYPE (type1)) != OFFSET_TYPE)
+      if (TREE_CODE (type1) == POINTER_TYPE)
        break;
     case NEGATE_EXPR:
       if (ARITHMETIC_TYPE_P (type1))
        break;
-      return candidates;
+      return;
 
 /* 11For every promoted integral type T,  there  exist  candidate  operator
      functions of the form
@@ -1685,7 +1650,7 @@ add_builtin_candidate (struct z_candidate *candidates, enum tree_code code,
     case BIT_NOT_EXPR:
       if (INTEGRAL_TYPE_P (type1))
        break;
-      return candidates;
+      return;
 
 /* 12For every quintuple C1, C2, T, CV1, CV2), where C2 is a class type, C1
      is the same type as C2 or is a derived class of C2, T  is  a  complete
@@ -1696,19 +1661,17 @@ add_builtin_candidate (struct z_candidate *candidates, enum tree_code code,
 
     case MEMBER_REF:
       if (TREE_CODE (type1) == POINTER_TYPE
-         && (TYPE_PTRMEMFUNC_P (type2) || TYPE_PTRMEM_P (type2)))
+         && TYPE_PTR_TO_MEMBER_P (type2))
        {
          tree c1 = TREE_TYPE (type1);
-         tree c2 = (TYPE_PTRMEMFUNC_P (type2)
-                    ? TYPE_METHOD_BASETYPE (TREE_TYPE (TYPE_PTRMEMFUNC_FN_TYPE (type2)))
-                    : TYPE_OFFSET_BASETYPE (TREE_TYPE (type2)));
+         tree c2 = TYPE_PTRMEM_CLASS_TYPE (type2);
 
          if (IS_AGGR_TYPE (c1) && DERIVED_FROM_P (c2, c1)
              && (TYPE_PTRMEMFUNC_P (type2)
                  || is_complete (TREE_TYPE (TREE_TYPE (type2)))))
            break;
        }
-      return candidates;
+      return;
 
 /* 13For every pair of promoted arithmetic types L and R, there exist  can-
      didate operator functions of the form
@@ -1764,26 +1727,24 @@ add_builtin_candidate (struct z_candidate *candidates, enum tree_code code,
     case TRUNC_DIV_EXPR:
       if (ARITHMETIC_TYPE_P (type1) && ARITHMETIC_TYPE_P (type2))
        break;
-      return candidates;
+      return;
 
     case EQ_EXPR:
     case NE_EXPR:
       if ((TYPE_PTRMEMFUNC_P (type1) && TYPE_PTRMEMFUNC_P (type2))
          || (TYPE_PTRMEM_P (type1) && TYPE_PTRMEM_P (type2)))
        break;
-      if ((TYPE_PTRMEMFUNC_P (type1) || TYPE_PTRMEM_P (type1))
-         && null_ptr_cst_p (args[1]))
+      if (TYPE_PTR_TO_MEMBER_P (type1) && null_ptr_cst_p (args[1]))
        {
          type2 = type1;
          break;
        }
-      if ((TYPE_PTRMEMFUNC_P (type2) || TYPE_PTRMEM_P (type2))
-         && null_ptr_cst_p (args[0]))
+      if (TYPE_PTR_TO_MEMBER_P (type2) && null_ptr_cst_p (args[0]))
        {
          type1 = type2;
          break;
        }
-      /* FALLTHROUGH */
+      /* Fall through.  */
     case LT_EXPR:
     case GT_EXPR:
     case LE_EXPR:
@@ -1806,7 +1767,7 @@ add_builtin_candidate (struct z_candidate *candidates, enum tree_code code,
          type1 = type2;
          break;
        }
-      return candidates;
+      return;
 
     case PLUS_EXPR:
       if (ARITHMETIC_TYPE_P (type1) && ARITHMETIC_TYPE_P (type2))
@@ -1822,7 +1783,7 @@ add_builtin_candidate (struct z_candidate *candidates, enum tree_code code,
          type2 = ptrdiff_type_node;
          break;
        }
-      return candidates;
+      return;
 
 /* 18For  every pair of promoted integral types L and R, there exist candi-
      date operator functions of the form
@@ -1843,7 +1804,7 @@ add_builtin_candidate (struct z_candidate *candidates, enum tree_code code,
     case RSHIFT_EXPR:
       if (INTEGRAL_TYPE_P (type1) && INTEGRAL_TYPE_P (type2))
        break;
-      return candidates;
+      return;
 
 /* 19For  every  triple  L, VQ, R), where L is an arithmetic or enumeration
      type, VQ is either volatile or empty, and R is a  promoted  arithmetic
@@ -1895,7 +1856,7 @@ add_builtin_candidate (struct z_candidate *candidates, enum tree_code code,
        case TRUNC_DIV_EXPR:
          if (ARITHMETIC_TYPE_P (type1) && ARITHMETIC_TYPE_P (type2))
            break;
-         return candidates;
+         return;
 
        case TRUNC_MOD_EXPR:
        case BIT_AND_EXPR:
@@ -1905,7 +1866,7 @@ add_builtin_candidate (struct z_candidate *candidates, enum tree_code code,
        case RSHIFT_EXPR:
          if (INTEGRAL_TYPE_P (type1) && INTEGRAL_TYPE_P (type2))
            break;
-         return candidates;
+         return;
 
        case NOP_EXPR:
          if (ARITHMETIC_TYPE_P (type1) && ARITHMETIC_TYPE_P (type2))
@@ -1920,7 +1881,7 @@ add_builtin_candidate (struct z_candidate *candidates, enum tree_code code,
              type2 = type1;
              break;
            }
-         return candidates;
+         return;
 
        default:
          abort ();
@@ -1949,22 +1910,15 @@ add_builtin_candidate (struct z_candidate *candidates, enum tree_code code,
        break;
 
       /* Otherwise, the types should be pointers.  */
-      if (!(TREE_CODE (type1) == POINTER_TYPE
-           || TYPE_PTRMEM_P (type1)
-           || TYPE_PTRMEMFUNC_P (type1))
-         || !(TREE_CODE (type2) == POINTER_TYPE
-              || TYPE_PTRMEM_P (type2)
-              || TYPE_PTRMEMFUNC_P (type2)))
-       return candidates;
+      if (!(TYPE_PTR_P (type1) || TYPE_PTR_TO_MEMBER_P (type1))
+         || !(TYPE_PTR_P (type2) || TYPE_PTR_TO_MEMBER_P (type2)))
+       return;
       
       /* We don't check that the two types are the same; the logic
         below will actually create two candidates; one in which both
         parameter types are TYPE1, and one in which both parameter
         types are TYPE2.  */
-       break;
-
-      /* These arguments do not make for a valid overloaded operator.  */
-      return candidates;
+      break;
 
     default:
       abort ();
@@ -1975,19 +1929,20 @@ add_builtin_candidate (struct z_candidate *candidates, enum tree_code code,
   if (type2 && !same_type_p (type1, type2)
       && TREE_CODE (type1) == TREE_CODE (type2)
       && (TREE_CODE (type1) == REFERENCE_TYPE
-         || (TREE_CODE (type1) == POINTER_TYPE
-             && TYPE_PTRMEM_P (type1) == TYPE_PTRMEM_P (type2))
+         || (TYPE_PTR_P (type1) && TYPE_PTR_P (type2))
+         || (TYPE_PTRMEM_P (type1) && TYPE_PTRMEM_P (type2))
          || TYPE_PTRMEMFUNC_P (type1)
          || IS_AGGR_TYPE (type1)
          || TREE_CODE (type1) == ENUMERAL_TYPE))
     {
-      candidates = build_builtin_candidate
+      build_builtin_candidate
        (candidates, fnname, type1, type1, args, argtypes, flags);
-      return build_builtin_candidate
+      build_builtin_candidate
        (candidates, fnname, type2, type2, args, argtypes, flags);
+      return;
     }
 
-  return build_builtin_candidate
+  build_builtin_candidate
     (candidates, fnname, type1, type2, args, argtypes, flags);
 }
 
@@ -2014,8 +1969,8 @@ type_decays_to (tree type)
    other cases which the standard disallows. add_builtin_candidate will
    filter out the invalid set.  */
 
-static struct z_candidate *
-add_builtin_candidates (struct z_candidate *candidates, enum tree_code code,
+static void
+add_builtin_candidates (struct z_candidate **candidates, enum tree_code code,
                         enum tree_code code2, tree fnname, tree *args,
                         int flags)
 {
@@ -2057,20 +2012,22 @@ add_builtin_candidates (struct z_candidate *candidates, enum tree_code code,
             bool    operator||(bool, bool);  */
 
     case TRUTH_NOT_EXPR:
-      return build_builtin_candidate
+      build_builtin_candidate
        (candidates, fnname, boolean_type_node,
         NULL_TREE, args, argtypes, flags);
+      return;
 
     case TRUTH_ORIF_EXPR:
     case TRUTH_ANDIF_EXPR:
-      return build_builtin_candidate
+      build_builtin_candidate
        (candidates, fnname, boolean_type_node,
         boolean_type_node, args, argtypes, flags);
+      return;
 
     case ADDR_EXPR:
     case COMPOUND_EXPR:
     case COMPONENT_REF:
-      return candidates;
+      return;
 
     case COND_EXPR:
     case EQ_EXPR:
@@ -2080,7 +2037,7 @@ add_builtin_candidates (struct z_candidate *candidates, enum tree_code code,
     case GT_EXPR:
     case GE_EXPR:
       enum_p = 1;
-      /* FALLTHROUGH */
+      /* Fall through.  */
     
     default:
       ref1 = 0;
@@ -2097,7 +2054,7 @@ add_builtin_candidates (struct z_candidate *candidates, enum tree_code code,
          tree convs;
 
          if (i == 0 && code == MODIFY_EXPR && code2 == NOP_EXPR)
-           return candidates;
+           return;
 
          convs = lookup_conversions (argtypes[i]);
 
@@ -2112,7 +2069,7 @@ add_builtin_candidates (struct z_candidate *candidates, enum tree_code code,
            }
 
          else if (! convs)
-           return candidates;
+           return;
 
          for (; convs; convs = TREE_CHAIN (convs))
            {
@@ -2164,16 +2121,16 @@ add_builtin_candidates (struct z_candidate *candidates, enum tree_code code,
     {
       if (types[1])
        for (type = types[1]; type; type = TREE_CHAIN (type))
-         candidates = add_builtin_candidate
+         add_builtin_candidate
            (candidates, code, code2, fnname, TREE_VALUE (types[0]),
             TREE_VALUE (type), args, argtypes, flags);
       else
-       candidates = add_builtin_candidate
+       add_builtin_candidate
          (candidates, code, code2, fnname, TREE_VALUE (types[0]),
           NULL_TREE, args, argtypes, flags);
     }
 
-  return candidates;
+  return;
 }
 
 
@@ -2188,7 +2145,7 @@ add_builtin_candidates (struct z_candidate *candidates, enum tree_code code,
    add_conv_candidate.  */
 
 static struct z_candidate*
-add_template_candidate_real (struct z_candidate *candidates, tree tmpl,
+add_template_candidate_real (struct z_candidate **candidates, tree tmpl,
                              tree ctype, tree explicit_targs, tree arglist,
                              tree return_type, tree access_path,
                             tree conversion_path, int flags, tree obj,
@@ -2216,11 +2173,11 @@ add_template_candidate_real (struct z_candidate *candidates, tree tmpl,
                           return_type, strict, -1);
 
   if (i != 0)
-    return candidates;
+    return NULL;
 
-  fn = instantiate_template (tmpl, targs);
+  fn = instantiate_template (tmpl, targs, tf_none);
   if (fn == error_mark_node)
-    return candidates;
+    return NULL;
 
   /* In [class.copy]:
 
@@ -2249,7 +2206,7 @@ add_template_candidate_real (struct z_candidate *candidates, tree tmpl,
       tree arg_types = FUNCTION_FIRST_USER_PARMTYPE (fn);
       if (arg_types && same_type_p (TYPE_MAIN_VARIANT (TREE_VALUE (arg_types)),
                                    ctype))
-       return candidates;
+       return NULL;
     }
 
   if (obj != NULL_TREE)
@@ -2287,7 +2244,7 @@ add_template_candidate_real (struct z_candidate *candidates, tree tmpl,
 
 
 static struct z_candidate *
-add_template_candidate (struct z_candidate *candidates, tree tmpl, tree ctype,
+add_template_candidate (struct z_candidate **candidates, tree tmpl, tree ctype,
                         tree explicit_targs, tree arglist, tree return_type,
                         tree access_path, tree conversion_path, int flags,
                         unification_kind_t strict)
@@ -2301,7 +2258,7 @@ add_template_candidate (struct z_candidate *candidates, tree tmpl, tree ctype,
 
 
 static struct z_candidate *
-add_template_conv_candidate (struct z_candidate *candidates, tree tmpl,
+add_template_conv_candidate (struct z_candidate **candidates, tree tmpl,
                              tree obj, tree arglist, tree return_type,
                             tree access_path, tree conversion_path)
 {
@@ -2311,14 +2268,42 @@ add_template_conv_candidate (struct z_candidate *candidates, tree tmpl,
                                 conversion_path, 0, obj, DEDUCE_CONV);
 }
 
+/* The CANDS are the set of candidates that were considered for
+   overload resolution.  Return the set of viable candidates.  If none
+   of the candidates were viable, set *ANY_VIABLE_P to true.  STRICT_P
+   is true if a candidate should be considered viable only if it is
+   strictly viable.  */
 
-static bool
-any_viable (struct z_candidate *cands)
+static struct z_candidate*
+splice_viable (struct z_candidate *cands,
+              bool strict_p,
+              bool *any_viable_p)
 {
-  for (; cands; cands = cands->next)
-    if (pedantic ? cands->viable == 1 : cands->viable)
-      return true;
-  return false;
+  struct z_candidate *viable;
+  struct z_candidate **last_viable;
+  struct z_candidate **cand;
+
+  viable = NULL;
+  last_viable = &viable;
+  *any_viable_p = false;
+
+  cand = &cands; 
+  while (*cand) 
+    {
+      struct z_candidate *c = *cand;
+      if (strict_p ? c->viable == 1 : c->viable)
+       {
+         *last_viable = c;
+         *cand = c->next;
+         c->next = NULL;
+         last_viable = &c->next;
+         *any_viable_p = true;
+       }
+      else
+       cand = &c->next;
+    }
+
+  return viable ? viable : cands;
 }
 
 static bool
@@ -2330,59 +2315,141 @@ any_strictly_viable (struct z_candidate *cands)
   return false;
 }
 
-static struct z_candidate *
-splice_viable (struct z_candidate *cands)
+static tree
+build_this (tree obj)
 {
-  struct z_candidate **p = &cands;
+  /* Fix this to work on non-lvalues.  */
+  return build_unary_op (ADDR_EXPR, obj, 0);
+}
 
-  for (; *p; )
-    {
-      if (pedantic ? (*p)->viable == 1 : (*p)->viable)
-       p = &((*p)->next);
-      else
-       *p = (*p)->next;
-    }
+/* Returns true iff functions are equivalent. Equivalent functions are
+   not '==' only if one is a function-local extern function or if
+   both are extern "C".  */
 
-  return cands;
+static inline int
+equal_functions (tree fn1, tree fn2)
+{
+  if (DECL_LOCAL_FUNCTION_P (fn1) || DECL_LOCAL_FUNCTION_P (fn2)
+      || DECL_EXTERN_C_FUNCTION_P (fn1))
+    return decls_match (fn1, fn2);
+  return fn1 == fn2;
 }
 
-static tree
-build_this (tree obj)
+/* Print information about one overload candidate CANDIDATE.  MSGSTR
+   is the text to print before the candidate itself.
+
+   NOTE: Unlike most diagnostic functions in GCC, MSGSTR is expected
+   to have been run through gettext by the caller.  This wart makes
+   life simpler in print_z_candidates and for the translators.  */
+
+static void
+print_z_candidate (const char *msgstr, struct z_candidate *candidate)
 {
-  /* Fix this to work on non-lvalues.  */
-  return build_unary_op (ADDR_EXPR, obj, 0);
+  if (TREE_CODE (candidate->fn) == IDENTIFIER_NODE)
+    {
+      if (TREE_VEC_LENGTH (candidate->convs) == 3)
+       inform ("%s %D(%T, %T, %T) <built-in>", msgstr, candidate->fn,
+               TREE_TYPE (TREE_VEC_ELT (candidate->convs, 0)),
+               TREE_TYPE (TREE_VEC_ELT (candidate->convs, 1)),
+               TREE_TYPE (TREE_VEC_ELT (candidate->convs, 2)));
+      else if (TREE_VEC_LENGTH (candidate->convs) == 2)
+       inform ("%s %D(%T, %T) <built-in>", msgstr, candidate->fn,
+               TREE_TYPE (TREE_VEC_ELT (candidate->convs, 0)),
+               TREE_TYPE (TREE_VEC_ELT (candidate->convs, 1)));
+      else
+       inform ("%s %D(%T) <built-in>", msgstr, candidate->fn,
+               TREE_TYPE (TREE_VEC_ELT (candidate->convs, 0)));
+    }
+  else if (TYPE_P (candidate->fn))
+    inform ("%s %T <conversion>", msgstr, candidate->fn);
+  else if (candidate->viable == -1)
+    inform ("%J%s %+#D <near match>", candidate->fn, msgstr, candidate->fn);
+  else
+    inform ("%J%s %+#D", candidate->fn, msgstr, candidate->fn);
 }
 
 static void
 print_z_candidates (struct z_candidate *candidates)
 {
-  const char *str = "candidates are:";
-  for (; candidates; candidates = candidates->next)
-    {
-      if (TREE_CODE (candidates->fn) == IDENTIFIER_NODE)
+  const char *str;
+  struct z_candidate *cand1;
+  struct z_candidate **cand2;
+
+  /* There may be duplicates in the set of candidates.  We put off
+     checking this condition as long as possible, since we have no way
+     to eliminate duplicates from a set of functions in less than n^2
+     time.  Now we are about to emit an error message, so it is more
+     permissible to go slowly.  */
+  for (cand1 = candidates; cand1; cand1 = cand1->next)
+    {
+      tree fn = cand1->fn;
+      /* Skip builtin candidates and conversion functions.  */
+      if (TREE_CODE (fn) != FUNCTION_DECL)
+       continue;
+      cand2 = &cand1->next;
+      while (*cand2)
        {
-         if (TREE_VEC_LENGTH (candidates->convs) == 3)
-           error ("%s %D(%T, %T, %T) <built-in>", str, candidates->fn,
-                     TREE_TYPE (TREE_VEC_ELT (candidates->convs, 0)),
-                     TREE_TYPE (TREE_VEC_ELT (candidates->convs, 1)),
-                     TREE_TYPE (TREE_VEC_ELT (candidates->convs, 2)));
-         else if (TREE_VEC_LENGTH (candidates->convs) == 2)
-           error ("%s %D(%T, %T) <built-in>", str, candidates->fn,
-                     TREE_TYPE (TREE_VEC_ELT (candidates->convs, 0)),
-                     TREE_TYPE (TREE_VEC_ELT (candidates->convs, 1)));
+         if (TREE_CODE ((*cand2)->fn) == FUNCTION_DECL
+             && equal_functions (fn, (*cand2)->fn))
+           *cand2 = (*cand2)->next;
          else
-           error ("%s %D(%T) <built-in>", str, candidates->fn,
-                     TREE_TYPE (TREE_VEC_ELT (candidates->convs, 0)));
+           cand2 = &(*cand2)->next;
        }
-      else if (TYPE_P (candidates->fn))
-       error ("%s %T <conversion>", str, candidates->fn);
-      else
-       cp_error_at ("%s %+#D%s", str, candidates->fn,
-                    candidates->viable == -1 ? " <near match>" : "");
-      str = "               "; 
+    }
+
+  if (!candidates)
+    return;
+
+  str = _("candidates are:");
+  print_z_candidate (str, candidates);
+  if (candidates->next)
+    {
+      /* Indent successive candidates by the width of the translation
+        of the above string.  */
+      size_t len = gcc_gettext_width (str) + 1;
+      char *spaces = alloca (len);
+      memset (spaces, ' ', len-1);
+      spaces[len - 1] = '\0';
+
+      candidates = candidates->next;
+      do
+       {
+         print_z_candidate (spaces, candidates);
+         candidates = candidates->next;
+       }
+      while (candidates);
     }
 }
 
+/* USER_SEQ is a user-defined conversion sequence, beginning with a
+   USER_CONV.  STD_SEQ is the standard conversion sequence applied to
+   the result of the conversion function to convert it to the final
+   desired type.  Merge the the two sequences into a single sequence,
+   and return the merged sequence.  */
+
+static tree
+merge_conversion_sequences (tree user_seq, tree std_seq)
+{
+  tree *t;
+
+  my_friendly_assert (TREE_CODE (user_seq) == USER_CONV,
+                     20030306);
+
+  /* Find the end of the second conversion sequence.  */
+  t = &(std_seq); 
+  while (TREE_CODE (*t) != IDENTITY_CONV)
+    t = &TREE_OPERAND (*t, 0);
+
+  /* Replace the identity conversion with the user conversion
+     sequence.  */
+  *t = user_seq;
+
+  /* The entire sequence is a user-conversion sequence.  */
+  ICS_USER_FLAG (std_seq) = 1;
+
+  return std_seq;
+}
+
 /* Returns the best overload candidate to perform the requested
    conversion.  This function is used for three the overloading situations
    described in [over.match.copy], [over.match.conv], and [over.match.ref].
@@ -2394,9 +2461,9 @@ build_user_type_conversion_1 (tree totype, tree expr, int flags)
 {
   struct z_candidate *candidates, *cand;
   tree fromtype = TREE_TYPE (expr);
-  tree ctors = NULL_TREE, convs = NULL_TREE, *p;
+  tree ctors = NULL_TREE, convs = NULL_TREE;
   tree args = NULL_TREE;
-  tree templates = NULL_TREE;
+  bool any_viable_p;
 
   /* We represent conversion within a hierarchy using RVALUE_CONV and
      BASE_CONV, as specified by [over.best.ics]; these become plain
@@ -2438,24 +2505,20 @@ build_user_type_conversion_1 (tree totype, tree expr, int flags)
        continue;
 
       if (TREE_CODE (ctor) == TEMPLATE_DECL) 
-       {
-         templates = tree_cons (NULL_TREE, ctor, templates);
-         candidates = 
-           add_template_candidate (candidates, ctor, totype,
-                                   NULL_TREE, args, NULL_TREE, 
-                                   TYPE_BINFO (totype),
-                                   TYPE_BINFO (totype),
-                                   flags,
-                                   DEDUCE_CALL);
-       }
+       cand = add_template_candidate (&candidates, ctor, totype,
+                                      NULL_TREE, args, NULL_TREE, 
+                                      TYPE_BINFO (totype),
+                                      TYPE_BINFO (totype),
+                                      flags,
+                                      DEDUCE_CALL);
       else 
-       candidates = add_function_candidate (candidates, ctor, totype,
-                                            args, TYPE_BINFO (totype), 
-                                            TYPE_BINFO (totype),
-                                            flags); 
+       cand = add_function_candidate (&candidates, ctor, totype,
+                                      args, TYPE_BINFO (totype), 
+                                      TYPE_BINFO (totype),
+                                      flags); 
 
-      if (candidates) 
-       candidates->second_conv = build1 (IDENTITY_CONV, totype, NULL_TREE);
+      if (cand)
+       cand->second_conv = build1 (IDENTITY_CONV, totype, NULL_TREE);
     }
 
   if (convs)
@@ -2477,7 +2540,6 @@ build_user_type_conversion_1 (tree totype, tree expr, int flags)
       for (fns = TREE_VALUE (convs); fns; fns = OVL_NEXT (fns))
        {
          tree fn = OVL_CURRENT (fns);
-         struct z_candidate *old_candidates = candidates;
          
          /* [over.match.funcs] For conversion functions, the function
             is considered to be a member of the class of the implicit
@@ -2487,44 +2549,41 @@ build_user_type_conversion_1 (tree totype, tree expr, int flags)
             So we pass fromtype as CTYPE to add_*_candidate.  */
 
          if (TREE_CODE (fn) == TEMPLATE_DECL)
-           {
-             templates = tree_cons (NULL_TREE, fn, templates);
-             candidates = 
-               add_template_candidate (candidates, fn, fromtype, NULL_TREE,
-                                       args, totype, TYPE_BINFO (fromtype), 
-                                       conversion_path,
-                                       flags,
-                                       DEDUCE_CONV);
-           } 
+           cand = add_template_candidate (&candidates, fn, fromtype, 
+                                          NULL_TREE,
+                                          args, totype, 
+                                          TYPE_BINFO (fromtype), 
+                                          conversion_path,
+                                          flags,
+                                          DEDUCE_CONV);
          else 
-           candidates = add_function_candidate (candidates, fn, fromtype,
-                                                args,
-                                                TYPE_BINFO (fromtype),
-                                                conversion_path,
-                                                flags); 
+           cand = add_function_candidate (&candidates, fn, fromtype,
+                                          args,
+                                          TYPE_BINFO (fromtype),
+                                          conversion_path,
+                                          flags); 
 
-         if (candidates != old_candidates)
+         if (cand)
            {
-             tree ics = implicit_conversion
-               (totype, TREE_TYPE (TREE_TYPE (candidates->fn)),
-                0, convflags);
+             tree ics = implicit_conversion (totype, 
+                                             TREE_TYPE (TREE_TYPE (cand->fn)),
+                                             0, convflags);
 
-             candidates->second_conv = ics;
+             cand->second_conv = ics;
              
              if (ics == NULL_TREE)
-               candidates->viable = 0;
+               cand->viable = 0;
              else if (candidates->viable == 1 && ICS_BAD_FLAG (ics))
-               candidates->viable = -1;
+               cand->viable = -1;
            }
        }
     }
 
-  if (! any_viable (candidates))
+  candidates = splice_viable (candidates, pedantic, &any_viable_p);
+  if (!any_viable_p)
     return 0;
 
-  candidates = splice_viable (candidates);
   cand = tourney (candidates);
-
   if (cand == 0)
     {
       if (flags & LOOKUP_COMPLAIN)
@@ -2537,24 +2596,29 @@ build_user_type_conversion_1 (tree totype, tree expr, int flags)
       cand = candidates;       /* any one will do */
       cand->second_conv = build1 (AMBIG_CONV, totype, expr);
       ICS_USER_FLAG (cand->second_conv) = 1;
-      /* Don't set ICS_BAD_FLAG; an ambiguous conversion is no worse than
-        another user-defined conversion.  */
+      if (!any_strictly_viable (candidates))
+       ICS_BAD_FLAG (cand->second_conv) = 1;
+      /* If there are viable candidates, don't set ICS_BAD_FLAG; an
+        ambiguous conversion is no worse than another user-defined
+        conversion.  */
 
       return cand;
     }
 
-  for (p = &(cand->second_conv); TREE_CODE (*p) != IDENTITY_CONV; )
-    p = &(TREE_OPERAND (*p, 0));
-
-  *p = build
+  /* Build the user conversion sequence.  */
+  convs = build_conv
     (USER_CONV,
      (DECL_CONSTRUCTOR_P (cand->fn)
       ? totype : non_reference (TREE_TYPE (TREE_TYPE (cand->fn)))),
-     expr, build_zc_wrapper (cand));
-  
-  ICS_USER_FLAG (cand->second_conv) = ICS_USER_FLAG (*p) = 1;
+     build1 (IDENTITY_CONV, TREE_TYPE (expr), expr));
+  TREE_OPERAND (convs, 1) = build_zc_wrapper (cand);
+
+  /* Combine it with the second conversion sequence.  */
+  cand->second_conv = merge_conversion_sequences (convs,
+                                                 cand->second_conv);
+
   if (cand->viable == -1)
-    ICS_BAD_FLAG (cand->second_conv) = ICS_BAD_FLAG (*p) = 1;
+    ICS_BAD_FLAG (cand->second_conv) = 1;
 
   return cand;
 }
@@ -2574,69 +2638,6 @@ build_user_type_conversion (tree totype, tree expr, int flags)
   return NULL_TREE;
 }
 
-/* Find the possibly overloaded set of functions corresponding to a
-   call of the form SCOPE::NAME (...). NAME might be a
-   TEMPLATE_ID_EXPR, OVERLOAD, _DECL, IDENTIFIER_NODE or LOOKUP_EXPR. */
-
-tree
-resolve_scoped_fn_name (tree scope, tree name)
-{
-  tree fn;
-  tree template_args = NULL_TREE;
-  bool is_template_id = TREE_CODE (name) == TEMPLATE_ID_EXPR;
-  
-  if (is_template_id)
-    {
-      template_args = TREE_OPERAND (name, 1);
-      name = TREE_OPERAND (name, 0);
-    }
-  if (TREE_CODE (name) == OVERLOAD)
-    name = DECL_NAME (get_first_fn (name));
-  else if (TREE_CODE (name) == LOOKUP_EXPR)
-    name = TREE_OPERAND (name, 0);
-  
-  if (TREE_CODE (scope) == NAMESPACE_DECL)
-    fn = lookup_namespace_name (scope, name);
-  else
-    {
-      if (!TYPE_BEING_DEFINED (scope)
-         && !COMPLETE_TYPE_P (complete_type (scope)))
-       {
-         error ("incomplete type '%T' cannot be used to name a scope",
-                scope);
-         return error_mark_node;
-       }
-      
-      if (BASELINK_P (name))
-       fn = name;
-      else
-       fn = lookup_member (scope, name, /*protect=*/1, /*prefer_type=*/0);
-      if (fn && current_class_type)
-       fn = (adjust_result_of_qualified_name_lookup 
-             (fn, scope, current_class_type));
-    }
-  
-  if (!fn)
-    {
-      error ("'%D' has no member named '%E'", scope, name);
-      return error_mark_node;
-    }
-  if (is_template_id)
-    {
-      tree fns = fn;
-
-      if (BASELINK_P (fn))
-       fns = BASELINK_FUNCTIONS (fns);
-      fns = build_nt (TEMPLATE_ID_EXPR, fns, template_args);
-      if (BASELINK_P (fn))
-       BASELINK_FUNCTIONS (fn) = fns;
-      else
-       fn = fns;
-    }
-  
-  return fn;
-}
-
 /* Do any initial processing on the arguments to a function call.  */
 
 static tree
@@ -2654,24 +2655,37 @@ resolve_args (tree args)
          error ("invalid use of void expression");
          return error_mark_node;
        }
-      else if (TREE_CODE (arg) == OFFSET_REF)
-       arg = resolve_offset_ref (arg);
       arg = convert_from_reference (arg);
       TREE_VALUE (t) = arg;
     }
   return args;
 }
 
-/* Return an expression for a call to FN (a namespace-scope function,
-   or a static member function) with the ARGS.  */
-      
-tree
-build_new_function_call (tree fn, tree args)
+/* Perform overload resolution on FN, which is called with the ARGS.
+
+   Return the candidate function selected by overload resolution, or
+   NULL if the event that overload resolution failed.  In the case
+   that overload resolution fails, *CANDIDATES will be the set of
+   candidates considered, and ANY_VIABLE_P will be set to true or
+   false to indicate whether or not any of the candidates were
+   viable.  
+
+   The ARGS should already have gone through RESOLVE_ARGS before this
+   function is called.  */
+
+static struct z_candidate *
+perform_overload_resolution (tree fn, 
+                            tree args, 
+                            struct z_candidate **candidates,
+                            bool *any_viable_p)
 {
-  struct z_candidate *candidates = 0, *cand;
+  struct z_candidate *cand;
   tree explicit_targs = NULL_TREE;
   int template_only = 0;
 
+  *candidates = NULL;
+  *any_viable_p = true;
+
   /* Check FN and ARGS.  */
   my_friendly_assert (TREE_CODE (fn) == FUNCTION_DECL 
                      || TREE_CODE (fn) == TEMPLATE_DECL
@@ -2688,67 +2702,142 @@ build_new_function_call (tree fn, tree args)
       template_only = 1;
     }
 
-  if (really_overloaded_fn (fn) 
-      || TREE_CODE (fn) == TEMPLATE_DECL)
-    {
-      tree t1;
-      tree templates = NULL_TREE;
+  /* Add the various candidate functions.  */
+  add_candidates (fn, args, explicit_targs, template_only,
+                 /*conversion_path=*/NULL_TREE,
+                 /*access_path=*/NULL_TREE,
+                 LOOKUP_NORMAL,
+                 candidates);
 
-      args = resolve_args (args);
+  *candidates = splice_viable (*candidates, pedantic, any_viable_p);
+  if (!*any_viable_p)
+    return NULL;
 
-      if (args == error_mark_node)
-       return error_mark_node;
+  cand = tourney (*candidates);
+  return cand;
+}
 
-      for (t1 = fn; t1; t1 = OVL_NEXT (t1))
-       {
-         tree t = OVL_CURRENT (t1);
+/* Return an expression for a call to FN (a namespace-scope function,
+   or a static member function) with the ARGS.  */
+      
+tree
+build_new_function_call (tree fn, tree args)
+{
+  struct z_candidate *candidates, *cand;
+  bool any_viable_p;
 
-         my_friendly_assert (!DECL_FUNCTION_MEMBER_P (t), 20020913);
+  args = resolve_args (args);
+  if (args == error_mark_node)
+    return error_mark_node;
 
-         if (TREE_CODE (t) == TEMPLATE_DECL)
-           {
-             templates = tree_cons (NULL_TREE, t, templates);
-             candidates = add_template_candidate
-               (candidates, t, NULL_TREE, explicit_targs, args, 
-                NULL_TREE, /*access_path=*/NULL_TREE, 
-                /*conversion_path=*/NULL_TREE,
-                LOOKUP_NORMAL, DEDUCE_CALL);  
-           }
-         else if (! template_only)
-           candidates = add_function_candidate
-             (candidates, t, NULL_TREE, args, 
-              /*access_path=*/NULL_TREE,
-              /*conversion_path=*/NULL_TREE, LOOKUP_NORMAL);
-       }
+  cand = perform_overload_resolution (fn, args, &candidates, &any_viable_p);
 
-      if (! any_viable (candidates))
-       {
-         if (candidates && ! candidates->next)
-           return build_function_call (candidates->fn, args);
-         error ("no matching function for call to `%D(%A)'",
-                   DECL_NAME (OVL_CURRENT (fn)), args);
-         if (candidates)
-           print_z_candidates (candidates);
-         return error_mark_node;
-       }
-      candidates = splice_viable (candidates);
-      cand = tourney (candidates);
+  if (!cand)
+    {
+      if (!any_viable_p && candidates && ! candidates->next)
+       return build_function_call (candidates->fn, args);
+      if (TREE_CODE (fn) == TEMPLATE_ID_EXPR)
+       fn = TREE_OPERAND (fn, 0);
+      if (!any_viable_p)
+       error ("no matching function for call to `%D(%A)'",
+              DECL_NAME (OVL_CURRENT (fn)), args);
+      else
+       error ("call of overloaded `%D(%A)' is ambiguous",
+              DECL_NAME (OVL_CURRENT (fn)), args);
+      if (candidates)
+       print_z_candidates (candidates);
+      return error_mark_node;
+    }
 
-      if (cand == 0)
-       {
-         error ("call of overloaded `%D(%A)' is ambiguous",
-                   DECL_NAME (OVL_FUNCTION (fn)), args);
-         print_z_candidates (candidates);
-         return error_mark_node;
-       }
+  return build_over_call (cand, LOOKUP_NORMAL);
+}
 
-      return build_over_call (cand, args, LOOKUP_NORMAL);
-    }
+/* Build a call to a global operator new.  FNNAME is the name of the
+   operator (either "operator new" or "operator new[]") and ARGS are
+   the arguments provided.  *SIZE points to the total number of bytes
+   required by the allocation, and is updated if that is changed here.
+   *COOKIE_SIZE is non-NULL if a cookie should be used.  If this
+   function determines that no cookie should be used, after all,
+   *COOKIE_SIZE is set to NULL_TREE.  */
 
-  /* This is not really overloaded.  */
-  fn = OVL_CURRENT (fn);
+tree
+build_operator_new_call (tree fnname, tree args, tree *size, tree *cookie_size)
+{
+  tree fns;
+  struct z_candidate *candidates;
+  struct z_candidate *cand;
+  bool any_viable_p;
 
-  return build_function_call (fn, args);
+  args = tree_cons (NULL_TREE, *size, args);
+  args = resolve_args (args);
+  if (args == error_mark_node)
+    return args;
+
+  fns = lookup_function_nonclass (fnname, args);
+
+  /* Figure out what function is being called.  */
+  cand = perform_overload_resolution (fns, args, &candidates, &any_viable_p);
+  
+  /* If no suitable function could be found, issue an error message
+     and give up.  */
+  if (!cand)
+    {
+      if (!any_viable_p)
+       error ("no matching function for call to `%D(%A)'",
+              DECL_NAME (OVL_CURRENT (fns)), args);
+      else
+       error ("call of overloaded `%D(%A)' is ambiguous",
+              DECL_NAME (OVL_CURRENT (fns)), args);
+      if (candidates)
+       print_z_candidates (candidates);
+      return error_mark_node;
+    }
+
+   /* If a cookie is required, add some extra space.  Whether
+      or not a cookie is required cannot be determined until
+      after we know which function was called.  */
+   if (*cookie_size)
+     {
+       bool use_cookie = true;
+       if (!abi_version_at_least (2))
+        {
+          tree placement = TREE_CHAIN (args);
+          /* In G++ 3.2, the check was implemented incorrectly; it
+             looked at the placement expression, rather than the
+             type of the function.  */
+          if (placement && !TREE_CHAIN (placement)
+              && same_type_p (TREE_TYPE (TREE_VALUE (placement)),
+                              ptr_type_node))
+            use_cookie = false;
+        }
+       else
+        {
+          tree arg_types;
+
+          arg_types = TYPE_ARG_TYPES (TREE_TYPE (cand->fn));
+          /* Skip the size_t parameter.  */
+          arg_types = TREE_CHAIN (arg_types);
+          /* Check the remaining parameters (if any).  */
+          if (arg_types 
+              && TREE_CHAIN (arg_types) == void_list_node
+              && same_type_p (TREE_VALUE (arg_types),
+                              ptr_type_node))
+            use_cookie = false;
+        }
+       /* If we need a cookie, adjust the number of bytes allocated.  */
+       if (use_cookie)
+        {
+          /* Update the total size.  */
+          *size = size_binop (PLUS_EXPR, *size, *cookie_size);
+          /* Update the argument list to reflect the adjusted size.  */
+          TREE_VALUE (args) = *size;
+        }
+       else
+        *cookie_size = NULL_TREE;
+     }
+
+   /* Build the CALL_EXPR.  */
+   return build_over_call (cand, LOOKUP_NORMAL);
 }
 
 static tree
@@ -2757,6 +2846,7 @@ build_object_call (tree obj, tree args)
   struct z_candidate *candidates = 0, *cand;
   tree fns, convs, mem_args = NULL_TREE;
   tree type = TREE_TYPE (obj);
+  bool any_viable_p;
 
   if (TYPE_PTRMEMFUNC_P (type))
     {
@@ -2784,17 +2874,14 @@ build_object_call (tree obj, tree args)
        {
          tree fn = OVL_CURRENT (fns);
          if (TREE_CODE (fn) == TEMPLATE_DECL)
-           {
-             candidates 
-               = add_template_candidate (candidates, fn, base, NULL_TREE,
-                                         mem_args, NULL_TREE, 
-                                         TYPE_BINFO (type),
-                                         TYPE_BINFO (type),
-                                         LOOKUP_NORMAL, DEDUCE_CALL);
-           }
+           add_template_candidate (&candidates, fn, base, NULL_TREE,
+                                   mem_args, NULL_TREE, 
+                                   TYPE_BINFO (type),
+                                   TYPE_BINFO (type),
+                                   LOOKUP_NORMAL, DEDUCE_CALL);
          else
-           candidates = add_function_candidate
-             (candidates, fn, base, mem_args, TYPE_BINFO (type),
+           add_function_candidate
+             (&candidates, fn, base, mem_args, TYPE_BINFO (type),
               TYPE_BINFO (type), LOOKUP_NORMAL);
        }
     }
@@ -2817,29 +2904,26 @@ build_object_call (tree obj, tree args)
          {
            tree fn = OVL_CURRENT (fns);
            if (TREE_CODE (fn) == TEMPLATE_DECL) 
-             {
-               candidates = (add_template_conv_candidate 
-                             (candidates, fn, obj, args, totype,
-                              /*access_path=*/NULL_TREE,
-                              /*conversion_path=*/NULL_TREE));
-             }
+             add_template_conv_candidate 
+               (&candidates, fn, obj, args, totype,
+                /*access_path=*/NULL_TREE,
+                /*conversion_path=*/NULL_TREE);
            else
-             candidates = add_conv_candidate (candidates, fn, obj, args,
-                                              /*conversion_path=*/NULL_TREE,
-                                              /*access_path=*/NULL_TREE);
+             add_conv_candidate (&candidates, fn, obj, args,
+                                 /*conversion_path=*/NULL_TREE,
+                                 /*access_path=*/NULL_TREE);
          }
     }
 
-  if (! any_viable (candidates))
+  candidates = splice_viable (candidates, pedantic, &any_viable_p);
+  if (!any_viable_p)
     {
       error ("no match for call to `(%T) (%A)'", TREE_TYPE (obj), args);
       print_z_candidates (candidates);
       return error_mark_node;
     }
 
-  candidates = splice_viable (candidates);
   cand = tourney (candidates);
-
   if (cand == 0)
     {
       error ("call of `(%T) (%A)' is ambiguous", TREE_TYPE (obj), args);
@@ -2852,7 +2936,7 @@ build_object_call (tree obj, tree args)
      DECL_NAME here.  */
   if (TREE_CODE (cand->fn) == FUNCTION_DECL
       && DECL_OVERLOADED_OPERATOR_P (cand->fn) == CALL_EXPR)
-    return build_over_call (cand, mem_args, LOOKUP_NORMAL);
+    return build_over_call (cand, LOOKUP_NORMAL);
 
   obj = convert_like_with_context
           (TREE_VEC_ELT (cand->convs, 0), obj, cand->fn, -1);
@@ -2875,23 +2959,32 @@ op_error (enum tree_code code, enum tree_code code2,
   switch (code)
     {
     case COND_EXPR:
-      error ("%s for `%T ? %T : %T' operator", problem,
-               error_type (arg1), error_type (arg2), error_type (arg3));
+      error ("%s for ternary 'operator?:' in '%E ? %E : %E'",
+             problem, arg1, arg2, arg3);
       break;
+      
     case POSTINCREMENT_EXPR:
     case POSTDECREMENT_EXPR:
-      error ("%s for `%T %s' operator", problem, error_type (arg1), opname);
+      error ("%s for 'operator%s' in '%E%s'", problem, opname, arg1, opname);
       break;
+      
     case ARRAY_REF:
-      error ("%s for `%T [%T]' operator", problem,
-               error_type (arg1), error_type (arg2));
+      error ("%s for 'operator[]' in '%E[%E]'", problem, arg1, arg2);
       break;
+
+    case REALPART_EXPR:
+    case IMAGPART_EXPR:
+      error ("%s for '%s' in '%s %E'", problem, opname, opname, arg1);
+      break;
+      
     default:
       if (arg2)
-       error ("%s for `%T %s %T' operator", problem,
-                 error_type (arg1), opname, error_type (arg2));
+       error ("%s for 'operator%s' in '%E %s %E'",
+               problem, opname, arg1, opname, arg2);
       else
-       error ("%s for `%s %T' operator", problem, opname, error_type (arg1));
+       error ("%s for 'operator%s' in '%s%E'",
+               problem, opname, opname, arg1);
+      break;
     }
 }
 
@@ -2904,6 +2997,7 @@ conditional_conversion (tree e1, tree e2)
   tree t1 = non_reference (TREE_TYPE (e1));
   tree t2 = non_reference (TREE_TYPE (e2));
   tree conv;
+  bool good_base;
 
   /* [expr.cond]
 
@@ -2930,29 +3024,36 @@ conditional_conversion (tree e1, tree e2)
      same cv-qualification as, or a greater cv-qualification than, the
      cv-qualification of T1.  If the conversion is applied, E1 is
      changed to an rvalue of type T2 that still refers to the original
-     source class object (or the appropriate subobject thereof).  */
+     source class object (or the appropriate subobject thereof).
+
+     FIXME we can't express an rvalue that refers to the original object;
+     we have to create a new one.  */
   if (CLASS_TYPE_P (t1) && CLASS_TYPE_P (t2)
-      && same_or_base_type_p (TYPE_MAIN_VARIANT (t2), 
-                             TYPE_MAIN_VARIANT (t1)))
+      && ((good_base = DERIVED_FROM_P (t2, t1)) || DERIVED_FROM_P (t1, t2)))
     {
-      if (at_least_as_qualified_p (t2, t1))
+      if (good_base && at_least_as_qualified_p (t2, t1))
        {
          conv = build1 (IDENTITY_CONV, t1, e1);
          if (!same_type_p (TYPE_MAIN_VARIANT (t1), 
                            TYPE_MAIN_VARIANT (t2)))
-           conv = build_conv (BASE_CONV, t2, conv);
+           {
+             conv = build_conv (BASE_CONV, t2, conv);
+             NEED_TEMPORARY_P (conv) = 1;
+           }
+         else
+           conv = build_conv (RVALUE_CONV, t2, conv);
          return conv;
        }
       else
        return NULL_TREE;
     }
+  else
+    /* [expr.cond]
 
-  /* [expr.cond]
-
-     E1 can be converted to match E2 if E1 can be implicitly converted
-     to the type that expression E2 would have if E2 were converted to
-     an rvalue (or the type it has, if E2 is an rvalue).  */
-  return implicit_conversion (t2, t1, e1, LOOKUP_NORMAL);
+       Otherwise: E1 can be converted to match E2 if E1 can be implicitly
+       converted to the type that expression E2 would have if E2 were
+       converted to an rvalue (or the type it has, if E2 is an rvalue).  */
+    return implicit_conversion (t2, t1, e1, LOOKUP_NORMAL);
 }
 
 /* Implement [expr.cond].  ARG1, ARG2, and ARG3 are the three
@@ -2989,16 +3090,13 @@ build_conditional_expr (tree arg1, tree arg2, tree arg3)
   
      The first expr ession is implicitly converted to bool (clause
      _conv_).  */
-  arg1 = cp_convert (boolean_type_node, arg1);
+  arg1 = perform_implicit_conversion (boolean_type_node, arg1);
 
   /* If something has already gone wrong, just pass that fact up the
      tree.  */
-  if (arg1 == error_mark_node 
-      || arg2 == error_mark_node 
-      || arg3 == error_mark_node 
-      || TREE_TYPE (arg1) == error_mark_node
-      || TREE_TYPE (arg2) == error_mark_node
-      || TREE_TYPE (arg3) == error_mark_node)
+  if (error_operand_p (arg1)
+      || error_operand_p (arg2)
+      || error_operand_p (arg3))
     return error_mark_node;
 
   /* [expr.cond]
@@ -3080,23 +3178,21 @@ build_conditional_expr (tree arg1, tree arg2, tree arg3)
        {
          arg2 = convert_like (conv2, arg2);
          arg2 = convert_from_reference (arg2);
-         /* That may not quite have done the trick.  If the two types
-            are cv-qualified variants of one another, we will have
-            just used an IDENTITY_CONV.  (There's no conversion from
-            an lvalue of one class type to an lvalue of another type,
-            even a cv-qualified variant, and we don't want to lose
-            lvalue-ness here.)  So, we manually add a NOP_EXPR here
-            if necessary.  */
-         if (!same_type_p (TREE_TYPE (arg2), arg3_type))
-           arg2 = build1 (NOP_EXPR, arg3_type, arg2);
+         if (!same_type_p (TREE_TYPE (arg2), arg3_type)
+             && CLASS_TYPE_P (arg3_type))
+           /* The types need to match if we're converting to a class type.
+              If not, we don't care about cv-qual mismatches, since
+              non-class rvalues are not cv-qualified.  */
+           abort ();
          arg2_type = TREE_TYPE (arg2);
        }
       else if (conv3 && !ICS_BAD_FLAG (conv3))
        {
          arg3 = convert_like (conv3, arg3);
          arg3 = convert_from_reference (arg3);
-         if (!same_type_p (TREE_TYPE (arg3), arg2_type))
-           arg3 = build1 (NOP_EXPR, arg2_type, arg3);
+         if (!same_type_p (TREE_TYPE (arg3), arg2_type)
+             && CLASS_TYPE_P (arg2_type))
+           abort ();
          arg3_type = TREE_TYPE (arg3);
        }
     }
@@ -3105,8 +3201,9 @@ build_conditional_expr (tree arg1, tree arg2, tree arg3)
 
      If the second and third operands are lvalues and have the same
      type, the result is of that type and is an lvalue.  */
-  if (real_lvalue_p (arg2) && real_lvalue_p (arg3) && 
-      same_type_p (arg2_type, arg3_type))
+  if (real_lvalue_p (arg2) 
+      && real_lvalue_p (arg3) 
+      && same_type_p (arg2_type, arg3_type))
     {
       result_type = arg2_type;
       goto valid_operands;
@@ -3125,6 +3222,7 @@ build_conditional_expr (tree arg1, tree arg2, tree arg3)
     {
       tree args[3];
       tree conv;
+      bool any_viable_p;
 
       /* Rearrange the arguments so that add_builtin_candidate only has
         to know about two args.  In build_builtin_candidates, the
@@ -3132,24 +3230,24 @@ build_conditional_expr (tree arg1, tree arg2, tree arg3)
       args[0] = arg2;
       args[1] = arg3;
       args[2] = arg1;
-      candidates = add_builtin_candidates (candidates, 
-                                          COND_EXPR, 
-                                          NOP_EXPR,
-                                          ansi_opname (COND_EXPR),
-                                          args,
-                                          LOOKUP_NORMAL);
+      add_builtin_candidates (&candidates, 
+                             COND_EXPR, 
+                             NOP_EXPR,
+                             ansi_opname (COND_EXPR),
+                             args,
+                             LOOKUP_NORMAL);
 
       /* [expr.cond]
 
         If the overload resolution fails, the program is
         ill-formed.  */
-      if (!any_viable (candidates))
+      candidates = splice_viable (candidates, pedantic, &any_viable_p);
+      if (!any_viable_p)
        {
          op_error (COND_EXPR, NOP_EXPR, arg1, arg2, arg3, "no match");
          print_z_candidates (candidates);
          return error_mark_node;
        }
-      candidates = splice_viable (candidates);
       cand = tourney (candidates);
       if (!cand)
        {
@@ -3185,18 +3283,10 @@ build_conditional_expr (tree arg1, tree arg2, tree arg3)
      We use ocp_convert rather than build_user_type_conversion because the
      latter returns NULL_TREE on failure, while the former gives an error.  */
 
-  if (IS_AGGR_TYPE (TREE_TYPE (arg2)) && real_lvalue_p (arg2))
-    arg2 = ocp_convert (TREE_TYPE (arg2), arg2,
-                       CONV_IMPLICIT|CONV_FORCE_TEMP, LOOKUP_NORMAL);
-  else
-    arg2 = decay_conversion (arg2);
+  arg2 = force_rvalue (arg2);
   arg2_type = TREE_TYPE (arg2);
 
-  if (IS_AGGR_TYPE (TREE_TYPE (arg3)) && real_lvalue_p (arg3))
-    arg3 = ocp_convert (TREE_TYPE (arg3), arg3,
-                       CONV_IMPLICIT|CONV_FORCE_TEMP, LOOKUP_NORMAL);
-  else
-    arg3 = decay_conversion (arg3);
+  arg3 = force_rvalue (arg3);
   arg3_type = TREE_TYPE (arg3);
 
   if (arg2 == error_mark_node || arg3 == error_mark_node)
@@ -3255,18 +3345,17 @@ build_conditional_expr (tree arg1, tree arg2, tree arg3)
        cv-qualification of either the second or the third operand.
        The result is of the common type.  */
   else if ((null_ptr_cst_p (arg2) 
-           && (TYPE_PTR_P (arg3_type) || TYPE_PTRMEM_P (arg3_type)
-               || TYPE_PTRMEMFUNC_P (arg3_type)))
+           && (TYPE_PTR_P (arg3_type) || TYPE_PTR_TO_MEMBER_P (arg3_type)))
           || (null_ptr_cst_p (arg3) 
-              && (TYPE_PTR_P (arg2_type) || TYPE_PTRMEM_P (arg2_type)
-               || TYPE_PTRMEMFUNC_P (arg2_type)))
+              && (TYPE_PTR_P (arg2_type) || TYPE_PTR_TO_MEMBER_P (arg2_type)))
           || (TYPE_PTR_P (arg2_type) && TYPE_PTR_P (arg3_type))
           || (TYPE_PTRMEM_P (arg2_type) && TYPE_PTRMEM_P (arg3_type))
-          || (TYPE_PTRMEMFUNC_P (arg2_type) 
-              && TYPE_PTRMEMFUNC_P (arg3_type)))
+          || (TYPE_PTRMEMFUNC_P (arg2_type) && TYPE_PTRMEMFUNC_P (arg3_type)))
     {
       result_type = composite_pointer_type (arg2_type, arg3_type, arg2,
                                            arg3, "conditional expression");
+      if (result_type == error_mark_node)
+       return error_mark_node;
       arg2 = perform_implicit_conversion (result_type, arg2);
       arg3 = perform_implicit_conversion (result_type, arg3);
     }
@@ -3279,43 +3368,124 @@ build_conditional_expr (tree arg1, tree arg2, tree arg3)
 
  valid_operands:
   result = fold (build (COND_EXPR, result_type, arg1, arg2, arg3));
+  /* We can't use result_type below, as fold might have returned a
+     throw_expr.  */
+
   /* Expand both sides into the same slot, hopefully the target of the
      ?: expression.  We used to check for TARGET_EXPRs here, but now we
      sometimes wrap them in NOP_EXPRs so the test would fail.  */
-  if (!lvalue_p && IS_AGGR_TYPE (result_type))
-    result = build_target_expr_with_type (result, result_type);
+  if (!lvalue_p && IS_AGGR_TYPE (TREE_TYPE (result)))
+    result = get_target_expr (result);
   
   /* If this expression is an rvalue, but might be mistaken for an
      lvalue, we must add a NON_LVALUE_EXPR.  */
   if (!lvalue_p && real_lvalue_p (result))
-    result = build1 (NON_LVALUE_EXPR, result_type, result);
+    result = build1 (NON_LVALUE_EXPR, TREE_TYPE (result), result);
 
   return result;
 }
 
+/* OPERAND is an operand to an expression.  Perform necessary steps
+   required before using it.  If OPERAND is NULL_TREE, NULL_TREE is
+   returned.  */
+
+static tree
+prep_operand (tree operand)
+{
+  if (operand)
+    {
+      operand = convert_from_reference (operand);
+      if (CLASS_TYPE_P (TREE_TYPE (operand))
+         && CLASSTYPE_TEMPLATE_INSTANTIATION (TREE_TYPE (operand)))
+       /* Make sure the template type is instantiated now.  */
+       instantiate_class_template (TYPE_MAIN_VARIANT (TREE_TYPE (operand)));
+    }
+
+  return operand;
+}
+
+/* Add each of the viable functions in FNS (a FUNCTION_DECL or
+   OVERLOAD) to the CANDIDATES, returning an updated list of
+   CANDIDATES.  The ARGS are the arguments provided to the call,
+   without any implicit object parameter.  The EXPLICIT_TARGS are
+   explicit template arguments provided.  TEMPLATE_ONLY is true if
+   only template functions should be considered.  CONVERSION_PATH,
+   ACCESS_PATH, and FLAGS are as for add_function_candidate.  */
+
+static void
+add_candidates (tree fns, tree args, 
+               tree explicit_targs, bool template_only,
+               tree conversion_path, tree access_path,
+               int flags,
+               struct z_candidate **candidates)
+{
+  tree ctype;
+  tree non_static_args;
+
+  ctype = conversion_path ? BINFO_TYPE (conversion_path) : NULL_TREE;
+  /* Delay creating the implicit this parameter until it is needed.  */
+  non_static_args = NULL_TREE;
+
+  while (fns) 
+    {
+      tree fn;
+      tree fn_args;
+
+      fn = OVL_CURRENT (fns);
+      /* Figure out which set of arguments to use.  */
+      if (DECL_NONSTATIC_MEMBER_FUNCTION_P (fn))
+       {
+         /* If this function is a non-static member, prepend the implicit
+            object parameter.  */
+         if (!non_static_args)
+           non_static_args = tree_cons (NULL_TREE,
+                                        build_this (TREE_VALUE (args)),
+                                        TREE_CHAIN (args));
+         fn_args = non_static_args;
+       }
+      else
+       /* Otherwise, just use the list of arguments provided.  */
+       fn_args = args;
+
+      if (TREE_CODE (fn) == TEMPLATE_DECL)
+       add_template_candidate (candidates, 
+                               fn, 
+                               ctype,
+                               explicit_targs,
+                               fn_args,
+                               NULL_TREE,
+                               access_path,
+                               conversion_path,
+                               flags,
+                               DEDUCE_CALL);
+      else if (!template_only)
+       add_function_candidate (candidates,
+                               fn,
+                               ctype,
+                               fn_args,
+                               access_path,
+                               conversion_path,
+                               flags);
+      fns = OVL_NEXT (fns);
+    }
+}
+
 tree
 build_new_op (enum tree_code code, int flags, tree arg1, tree arg2, tree arg3)
 {
   struct z_candidate *candidates = 0, *cand;
-  tree fns, mem_arglist = NULL_TREE, arglist, fnname;
+  tree arglist, fnname;
+  tree args[3];
   enum tree_code code2 = NOP_EXPR;
-  tree templates = NULL_TREE;
   tree conv;
-  bool viable_candidates;
+  bool strict_p;
+  bool any_viable_p;
 
   if (error_operand_p (arg1) 
       || error_operand_p (arg2) 
       || error_operand_p (arg3))
     return error_mark_node;
 
-  /* This can happen if a template takes all non-type parameters, e.g.
-     undeclared_template<1, 5, 72>a;  */
-  if (code == LT_EXPR && TREE_CODE (arg1) == TEMPLATE_DECL)
-    {
-      error ("`%D' must be declared before use", arg1);
-      return error_mark_node;
-    }
-
   if (code == MODIFY_EXPR)
     {
       code2 = TREE_CODE (arg3);
@@ -3325,13 +3495,7 @@ build_new_op (enum tree_code code, int flags, tree arg1, tree arg2, tree arg3)
   else
     fnname = ansi_opname (code);
 
-  if (TREE_CODE (arg1) == OFFSET_REF)
-    arg1 = resolve_offset_ref (arg1);
-  arg1 = convert_from_reference (arg1);
-  if (CLASS_TYPE_P (TREE_TYPE (arg1))
-      && CLASSTYPE_TEMPLATE_INSTANTIATION (TREE_TYPE (arg1)))
-    /* Make sure the template type is instantiated now.  */
-    instantiate_class_template (TYPE_MAIN_VARIANT (TREE_TYPE (arg1)));
+  arg1 = prep_operand (arg1);
   
   switch (code)
     {
@@ -3349,24 +3513,8 @@ build_new_op (enum tree_code code, int flags, tree arg1, tree arg2, tree arg3)
       break;
     }
 
-  if (arg2)
-    {
-      if (TREE_CODE (arg2) == OFFSET_REF)
-       arg2 = resolve_offset_ref (arg2);
-      arg2 = convert_from_reference (arg2);
-      if (CLASS_TYPE_P (TREE_TYPE (arg2))
-         && CLASSTYPE_TEMPLATE_INSTANTIATION (TREE_TYPE (arg2)))
-       instantiate_class_template (TYPE_MAIN_VARIANT (TREE_TYPE (arg2)));
-    }
-  if (arg3)
-    {
-      if (TREE_CODE (arg3) == OFFSET_REF)
-       arg3 = resolve_offset_ref (arg3);
-      arg3 = convert_from_reference (arg3);
-      if (CLASS_TYPE_P (TREE_TYPE (arg3))
-         && CLASSTYPE_TEMPLATE_INSTANTIATION (TREE_TYPE (arg3)))
-       instantiate_class_template (TYPE_MAIN_VARIANT (TREE_TYPE (arg3)));
-    }
+  arg2 = prep_operand (arg2);
+  arg3 = prep_operand (arg3);
   
   if (code == COND_EXPR)
     {
@@ -3391,98 +3539,45 @@ build_new_op (enum tree_code code, int flags, tree arg1, tree arg2, tree arg3)
     arglist = tree_cons (NULL_TREE, arg2, arglist);
   arglist = tree_cons (NULL_TREE, arg1, arglist);
 
-  fns = lookup_function_nonclass (fnname, arglist);
-
-  if (fns && TREE_CODE (fns) == TREE_LIST)
-    fns = TREE_VALUE (fns);
-  for (; fns; fns = OVL_NEXT (fns))
+  /* Add namespace-scope operators to the list of functions to
+     consider.  */
+  add_candidates (lookup_function_nonclass (fnname, arglist),
+                 arglist, NULL_TREE, false, NULL_TREE, NULL_TREE,
+                 flags, &candidates);
+  /* Add class-member operators to the candidate set.  */
+  if (CLASS_TYPE_P (TREE_TYPE (arg1)))
     {
-      tree fn = OVL_CURRENT (fns);
-      if (TREE_CODE (fn) == TEMPLATE_DECL)
-       {
-         templates = tree_cons (NULL_TREE, fn, templates);
-         candidates 
-           = add_template_candidate (candidates, fn, NULL_TREE, NULL_TREE,
-                                     arglist, TREE_TYPE (fnname),
-                                     /*access_path=*/NULL_TREE,
-                                     /*conversion_path=*/NULL_TREE,
-                                     flags, DEDUCE_CALL); 
-       }
-      else
-       candidates = add_function_candidate (candidates, fn, NULL_TREE,
-                                            arglist,
-                                            /*access_path=*/NULL_TREE,
-                                            /*conversion_path=*/NULL_TREE,
-                                            flags);
-    }
+      tree fns;
 
-  if (IS_AGGR_TYPE (TREE_TYPE (arg1)))
-    {
       fns = lookup_fnfields (TYPE_BINFO (TREE_TYPE (arg1)), fnname, 1);
       if (fns == error_mark_node)
        return fns;
+      if (fns)
+       add_candidates (BASELINK_FUNCTIONS (fns), arglist, 
+                       NULL_TREE, false,
+                       BASELINK_BINFO (fns),
+                       TYPE_BINFO (TREE_TYPE (arg1)),
+                       flags, &candidates);
+    }
+
+  /* Rearrange the arguments for ?: so that add_builtin_candidate only has
+     to know about two args; a builtin candidate will always have a first
+     parameter of type bool.  We'll handle that in
+     build_builtin_candidate.  */
+  if (code == COND_EXPR)
+    {
+      args[0] = arg2;
+      args[1] = arg3;
+      args[2] = arg1;
     }
   else
-    fns = NULL_TREE;
-
-  if (fns)
     {
-      tree conversion_path = BASELINK_BINFO (fns);
-
-      mem_arglist = tree_cons (NULL_TREE, build_this (arg1), TREE_CHAIN (arglist));
-      for (fns = BASELINK_FUNCTIONS (fns); fns; fns = OVL_NEXT (fns))
-       {
-         tree fn = OVL_CURRENT (fns);
-         tree this_arglist;
-         tree access_path = TYPE_BINFO (TREE_TYPE (arg1));
-
-         if (TREE_CODE (TREE_TYPE (fn)) == METHOD_TYPE)
-           this_arglist = mem_arglist;
-         else
-           this_arglist = arglist;
-
-         if (TREE_CODE (fn) == TEMPLATE_DECL)
-           {
-             /* A member template.  */
-             templates = tree_cons (NULL_TREE, fn, templates);
-             candidates 
-               = add_template_candidate (candidates, fn, 
-                                         BINFO_TYPE (conversion_path),
-                                         NULL_TREE,
-                                         this_arglist,  TREE_TYPE (fnname),
-                                         access_path, conversion_path,
-                                         flags, DEDUCE_CALL); 
-           }
-         else
-           candidates = add_function_candidate
-             (candidates, fn, BINFO_TYPE (conversion_path), this_arglist, 
-              access_path, conversion_path, flags);
-       }
+      args[0] = arg1;
+      args[1] = arg2;
+      args[2] = NULL_TREE;
     }
 
-  {
-    tree args[3];
-
-    /* Rearrange the arguments for ?: so that add_builtin_candidate only has
-       to know about two args; a builtin candidate will always have a first
-       parameter of type bool.  We'll handle that in
-       build_builtin_candidate.  */
-    if (code == COND_EXPR)
-      {
-       args[0] = arg2;
-       args[1] = arg3;
-       args[2] = arg1;
-      }
-    else
-      {
-       args[0] = arg1;
-       args[1] = arg2;
-       args[2] = NULL_TREE;
-      }
-
-    candidates = add_builtin_candidates
-      (candidates, code, code2, fnname, args, flags);
-  }
+  add_builtin_candidates (&candidates, code, code2, fnname, args, flags);
 
   switch (code)
     {
@@ -3494,15 +3589,16 @@ build_new_op (enum tree_code code, int flags, tree arg1, tree arg2, tree arg3)
         operators.  The built-in candidate set for COMPONENT_REF
         would be empty too, but since there are no such built-in
         operators, we accept non-strict matches for them.  */
-      viable_candidates = any_strictly_viable (candidates);
+      strict_p = true;
       break;
 
     default:
-      viable_candidates = any_viable (candidates);
+      strict_p = pedantic;
       break;
     }      
 
-  if (! viable_candidates)
+  candidates = splice_viable (candidates, strict_p, &any_viable_p);
+  if (!any_viable_p)
     {
       switch (code)
        {
@@ -3536,9 +3632,8 @@ build_new_op (enum tree_code code, int flags, tree arg1, tree arg2, tree arg3)
        }
       return error_mark_node;
     }
-  candidates = splice_viable (candidates);
-  cand = tourney (candidates);
 
+  cand = tourney (candidates);
   if (cand == 0)
     {
       if (flags & LOOKUP_COMPLAIN)
@@ -3551,7 +3646,6 @@ build_new_op (enum tree_code code, int flags, tree arg1, tree arg2, tree arg3)
 
   if (TREE_CODE (cand->fn) == FUNCTION_DECL)
     {
-      extern int warn_synth;
       if (warn_synth
          && fnname == ansi_assopname (NOP_EXPR)
          && DECL_ARTIFICIAL (cand->fn)
@@ -3566,11 +3660,7 @@ build_new_op (enum tree_code code, int flags, tree arg1, tree arg2, tree arg3)
                         : candidates->fn);
        }
 
-      return build_over_call
-       (cand,
-        TREE_CODE (TREE_TYPE (cand->fn)) == METHOD_TYPE
-        ? mem_arglist : arglist,
-        LOOKUP_NORMAL);
+      return build_over_call (cand, LOOKUP_NORMAL);
     }
 
   /* Check for comparison of different enum types.  */
@@ -3699,15 +3789,13 @@ build_op_delete_call (enum tree_code code, tree addr, tree size,
                       int flags, tree placement)
 {
   tree fn = NULL_TREE;
-  tree fns, fnname, fntype, argtypes, args, type;
+  tree fns, fnname, argtypes, args, type;
   int pass;
 
   if (addr == error_mark_node)
     return error_mark_node;
 
-  type = TREE_TYPE (TREE_TYPE (addr));
-  while (TREE_CODE (type) == ARRAY_TYPE)
-    type = TREE_TYPE (type);
+  type = strip_array_types (TREE_TYPE (TREE_TYPE (addr)));
 
   fnname = ansi_opname (code);
 
@@ -3718,7 +3806,7 @@ build_op_delete_call (enum tree_code code, tree addr, tree size,
        the lookup selects a placement deallocation function, the
        program is ill-formed.
   
-       Therefore, we ask lookup_fnfields to complain ambout ambiguity.  */
+       Therefore, we ask lookup_fnfields to complain about ambiguity.  */
     {
       fns = lookup_fnfields (TYPE_BINFO (type), fnname, 1);
       if (fns == error_mark_node)
@@ -3737,10 +3825,6 @@ build_op_delete_call (enum tree_code code, tree addr, tree size,
 
       /* Find the allocation function that is being called.  */
       call_expr = placement;
-      /* Sometimes we have a COMPOUND_EXPR, rather than a simple
-        CALL_EXPR.  */
-      while (TREE_CODE (call_expr) == COMPOUND_EXPR)
-       call_expr = TREE_OPERAND (call_expr, 1);
       /* Extract the function.  */
       alloc_fn = get_callee_fndecl (call_expr);
       my_friendly_assert (alloc_fn != NULL_TREE, 20020327);
@@ -3760,21 +3844,11 @@ build_op_delete_call (enum tree_code code, tree addr, tree size,
   addr = cp_convert (ptr_type_node, addr);
 
   /* We make two tries at finding a matching `operator delete'.  On
-     the first pass, we look for an one-operator (or placement)
+     the first pass, we look for a one-operator (or placement)
      operator delete.  If we're not doing placement delete, then on
      the second pass we look for a two-argument delete.  */
   for (pass = 0; pass < (placement ? 1 : 2); ++pass) 
     {
-      if (pass == 0)
-       argtypes = tree_cons (NULL_TREE, ptr_type_node, argtypes);
-      else 
-       /* Normal delete; now try to find a match including the size
-          argument.  */
-       argtypes = tree_cons (NULL_TREE, ptr_type_node,
-                             tree_cons (NULL_TREE, sizetype, 
-                                        void_list_node));
-      fntype = build_function_type (void_type_node, argtypes);
-
       /* Go through the `operator delete' functions looking for one
         with a matching type.  */
       for (fn = BASELINK_P (fns) ? BASELINK_FUNCTIONS (fns) : fns; 
@@ -3783,13 +3857,30 @@ build_op_delete_call (enum tree_code code, tree addr, tree size,
        {
          tree t;
 
-         /* Exception specifications on the `delete' operator do not
-            matter.  */
-         t = build_exception_variant (TREE_TYPE (OVL_CURRENT (fn)),
-                                      NULL_TREE);
-         /* We also don't compare attributes.  We're really just
-            trying to check the types of the first two parameters.  */
-         if (comptypes (t, fntype, COMPARE_NO_ATTRIBUTES))
+         /* The first argument must be "void *".  */
+         t = TYPE_ARG_TYPES (TREE_TYPE (OVL_CURRENT (fn)));
+         if (!same_type_p (TREE_VALUE (t), ptr_type_node))
+           continue;
+         t = TREE_CHAIN (t);
+         /* On the first pass, check the rest of the arguments.  */
+         if (pass == 0)
+           {
+             while (argtypes && t)
+               {
+                 if (!same_type_p (TREE_VALUE (argtypes),
+                                   TREE_VALUE (t)))
+                   break;
+                 argtypes = TREE_CHAIN (argtypes);
+                 t = TREE_CHAIN (t);
+               }
+             if (!argtypes && !t)
+               break;
+           }
+         /* On the second pass, the second argument must be
+            "size_t".  */
+         else if (pass == 1
+                  && same_type_p (TREE_VALUE (t), sizetype)
+                  && TREE_CHAIN (t) == void_list_node)
            break;
        }
 
@@ -3808,7 +3899,7 @@ build_op_delete_call (enum tree_code code, tree addr, tree size,
       /* If the FN is a member function, make sure that it is
         accessible.  */
       if (DECL_CLASS_SCOPE_P (fn))
-       enforce_access (type, fn);
+       perform_or_defer_access_check (TYPE_BINFO (type), fn);
 
       if (pass == 0)
        args = tree_cons (NULL_TREE, addr, args);
@@ -3816,7 +3907,15 @@ build_op_delete_call (enum tree_code code, tree addr, tree size,
        args = tree_cons (NULL_TREE, addr, 
                          build_tree_list (NULL_TREE, size));
 
-      return build_function_call (fn, args);
+      if (placement)
+       {
+         /* The placement args might not be suitable for overload
+            resolution at this point, so build the call directly.  */
+         mark_used (fn);
+         return build_cxx_call (fn, args, args);
+       }
+      else
+       return build_function_call (fn, args);
     }
 
   /* If we are doing placement delete we do nothing if we don't find a
@@ -3824,7 +3923,8 @@ build_op_delete_call (enum tree_code code, tree addr, tree size,
   if (placement)
     return NULL_TREE;
 
-  error ("no suitable `operator delete' for `%T'", type);
+  error ("no suitable `operator %s' for `%T'",
+        operator_name_info[(int)code].name, type);
   return error_mark_node;
 }
 
@@ -3835,6 +3935,8 @@ build_op_delete_call (enum tree_code code, tree addr, tree size,
 bool
 enforce_access (tree basetype_path, tree decl)
 {
+  my_friendly_assert (TREE_CODE (basetype_path) == TREE_VEC, 20030624);
+  
   if (!accessible_p (basetype_path, decl))
     {
       if (TREE_PRIVATE (decl))
@@ -3850,14 +3952,17 @@ enforce_access (tree basetype_path, tree decl)
   return true;
 }
 
-/* Perform the conversions in CONVS on the expression EXPR. 
-   FN and ARGNUM are used for diagnostics.  ARGNUM is zero based, -1
+/* Perform the conversions in CONVS on the expression EXPR.  FN and
+   ARGNUM are used for diagnostics.  ARGNUM is zero based, -1
    indicates the `this' argument of a method.  INNER is nonzero when
    being called to continue a conversion chain. It is negative when a
-   reference binding will be applied, positive otherwise.  */
+   reference binding will be applied, positive otherwise.  If
+   ISSUE_CONVERSION_WARNINGS is true, warnings about suspicious
+   conversions will be emitted if appropriate.  */
 
 static tree
-convert_like_real (tree convs, tree expr, tree fn, int argnum, int inner)
+convert_like_real (tree convs, tree expr, tree fn, int argnum, int inner,
+                  bool issue_conversion_warnings)
 {
   int savew, savee;
 
@@ -3873,11 +3978,13 @@ convert_like_real (tree convs, tree expr, tree fn, int argnum, int inner)
        {
          if (TREE_CODE (t) == USER_CONV || !ICS_BAD_FLAG (t))
            {
-             expr = convert_like_real (t, expr, fn, argnum, 1);
+             expr = convert_like_real (t, expr, fn, argnum, 1,
+                                       /*issue_conversion_warnings=*/false);
              break;
            }
          else if (TREE_CODE (t) == AMBIG_CONV)
-           return convert_like_real (t, expr, fn, argnum, 1);
+           return convert_like_real (t, expr, fn, argnum, 1,
+                                     /*issue_conversion_warnings=*/false);
          else if (TREE_CODE (t) == IDENTITY_CONV)
            break;
        }
@@ -3887,15 +3994,14 @@ convert_like_real (tree convs, tree expr, tree fn, int argnum, int inner)
       return cp_convert (totype, expr);
     }
   
-  if (!inner)
+  if (issue_conversion_warnings)
     expr = dubious_conversion_warnings
              (totype, expr, "argument", fn, argnum);
   switch (TREE_CODE (convs))
     {
     case USER_CONV:
       {
-       struct z_candidate *cand
-         = WRAPPER_ZC (TREE_OPERAND (convs, 1));
+       struct z_candidate *cand = USER_CONV_CAND (convs);
        tree convfn = cand->fn;
        tree args;
 
@@ -3914,7 +4020,7 @@ convert_like_real (tree convs, tree expr, tree fn, int argnum, int inner)
          }
        else
          args = build_this (expr);
-       expr = build_over_call (cand, args, LOOKUP_NORMAL);
+       expr = build_over_call (cand, LOOKUP_NORMAL);
 
        /* If this is a constructor or a function returning an aggr type,
           we need to build up a TARGET_EXPR.  */
@@ -3986,7 +4092,8 @@ convert_like_real (tree convs, tree expr, tree fn, int argnum, int inner)
     };
 
   expr = convert_like_real (TREE_OPERAND (convs, 0), expr, fn, argnum,
-                            TREE_CODE (convs) == REF_BIND ? -1 : 1);
+                            TREE_CODE (convs) == REF_BIND ? -1 : 1,
+                           /*issue_conversion_warnings=*/false);
   if (expr == error_mark_node)
     return error_mark_node;
 
@@ -3995,7 +4102,7 @@ convert_like_real (tree convs, tree expr, tree fn, int argnum, int inner)
     case RVALUE_CONV:
       if (! IS_AGGR_TYPE (totype))
        return expr;
-      /* else fall through */
+      /* Else fall through.  */
     case BASE_CONV:
       if (TREE_CODE (convs) == BASE_CONV && !NEED_TEMPORARY_P (convs))
        {
@@ -4032,9 +4139,26 @@ convert_like_real (tree convs, tree expr, tree fn, int argnum, int inner)
        tree ref_type = totype;
 
        /* If necessary, create a temporary.  */
-       if (NEED_TEMPORARY_P (convs) || !non_cast_lvalue_p (expr))
+       if (NEED_TEMPORARY_P (convs) || !lvalue_p (expr))
          {
            tree type = TREE_TYPE (TREE_OPERAND (convs, 0));
+
+           if (!CP_TYPE_CONST_NON_VOLATILE_P (TREE_TYPE (ref_type)))
+             {
+               /* If the reference is volatile or non-const, we
+                  cannot create a temporary.  */
+               cp_lvalue_kind lvalue = real_lvalue_p (expr);
+               
+               if (lvalue & clk_bitfield)
+                 error ("cannot bind bitfield `%E' to `%T'",
+                        expr, ref_type);
+               else if (lvalue & clk_packed)
+                 error ("cannot bind packed field `%E' to `%T'",
+                        expr, ref_type);
+               else
+                 error ("cannot bind rvalue `%E' to `%T'", expr, ref_type);
+               return error_mark_node;
+             }
            expr = build_target_expr_with_type (expr, type);
          }
 
@@ -4050,9 +4174,7 @@ convert_like_real (tree convs, tree expr, tree fn, int argnum, int inner)
        expr = cp_convert (build_pointer_type (TREE_TYPE (ref_type)), 
                           expr);
        /* Convert the pointer to the desired reference type.  */
-       expr = build1 (NOP_EXPR, ref_type, expr);
-
-       return expr;
+       return build_nop (ref_type, expr);
       }
 
     case LVALUE_CONV:
@@ -4070,37 +4192,45 @@ convert_like_real (tree convs, tree expr, tree fn, int argnum, int inner)
                      LOOKUP_NORMAL|LOOKUP_NO_CONVERSION);
 }
 
-/* Build a call to __builtin_trap which can be used in an expression.  */
+/* Build a call to __builtin_trap which can be used as an expression of
+   type TYPE.  */
 
 static tree
-call_builtin_trap (void)
+call_builtin_trap (tree type)
 {
-  tree fn = get_identifier ("__builtin_trap");
-  if (IDENTIFIER_GLOBAL_VALUE (fn))
-    fn = IDENTIFIER_GLOBAL_VALUE (fn);
-  else
-    abort ();
+  tree fn = IDENTIFIER_GLOBAL_VALUE (get_identifier ("__builtin_trap"));
 
+  my_friendly_assert (fn != NULL, 20030927);
   fn = build_call (fn, NULL_TREE);
-  fn = build (COMPOUND_EXPR, integer_type_node, fn, integer_zero_node);
+  fn = build (COMPOUND_EXPR, type, fn, error_mark_node);
+  fn = force_target_expr (type, fn);
   return fn;
 }
 
 /* ARG is being passed to a varargs function.  Perform any conversions
-   required.  Array/function to pointer decay must have already happened.
-   Return the converted value.  */
+   required.  Return the converted value.  */
 
 tree
 convert_arg_to_ellipsis (tree arg)
 {
+  /* [expr.call]
+
+     The lvalue-to-rvalue, array-to-pointer, and function-to-pointer
+     standard conversions are performed.  */
+  arg = decay_conversion (arg);
+  /* [expr.call]
+
+     If the argument has integral or enumeration type that is subject
+     to the integral promotions (_conv.prom_), or a floating point
+     type that is subject to the floating point promotion
+     (_conv.fpprom_), the value of the argument is converted to the
+     promoted type before the call.  */
   if (TREE_CODE (TREE_TYPE (arg)) == REAL_TYPE
       && (TYPE_PRECISION (TREE_TYPE (arg))
          < TYPE_PRECISION (double_type_node)))
-    /* Convert `float' to `double'.  */
-    arg = cp_convert (double_type_node, arg);
-  else
-    /* Convert `short' and `char' to full-size `int'.  */
-    arg = default_conversion (arg);
+    arg = convert_to_real (double_type_node, arg);
+  else if (INTEGRAL_OR_ENUMERATION_TYPE_P (TREE_TYPE (arg)))
+    arg = perform_integral_promotions (arg);
 
   arg = require_complete_type (arg);
   
@@ -4112,7 +4242,7 @@ convert_arg_to_ellipsis (tree arg)
       warning ("cannot pass objects of non-POD type `%#T' through `...'; \
 call will abort at runtime",
               TREE_TYPE (arg));
-      arg = call_builtin_trap ();
+      arg = call_builtin_trap (TREE_TYPE (arg));
     }
 
   return arg;
@@ -4134,8 +4264,10 @@ build_x_va_arg (tree expr, tree type)
   if (! pod_type_p (type))
     {
       /* Undefined behavior [expr.call] 5.2.2/7.  */
-      warning ("cannot receive objects of non-POD type `%#T' through `...'",
-                 type);
+      warning ("cannot receive objects of non-POD type `%#T' through `...'; \
+call will abort at runtime",
+              type);
+      return call_builtin_trap (type);
     }
   
   return build_va_arg (expr, type);
@@ -4150,11 +4282,9 @@ cxx_type_promotes_to (tree type)
 {
   tree promote;
 
-  if (TREE_CODE (type) == ARRAY_TYPE)
-    return build_pointer_type (TREE_TYPE (type));
-
-  if (TREE_CODE (type) == FUNCTION_TYPE)
-    return build_pointer_type (type);
+  /* Perform the array-to-pointer and function-to-pointer
+     conversions.  */
+  type = type_decays_to (type);
 
   promote = type_promotes_to (type);
   if (same_type_p (type, promote))
@@ -4216,7 +4346,9 @@ type_passed_as (tree type)
     type = build_reference_type (type);
   else if (PROMOTE_PROTOTYPES
           && INTEGRAL_TYPE_P (type)
-          && TYPE_PRECISION (type) < TYPE_PRECISION (integer_type_node))
+          && COMPLETE_TYPE_P (type)
+          && INT_CST_LT_UNSIGNED (TYPE_SIZE (type),
+                                  TYPE_SIZE (integer_type_node)))
     type = integer_type_node;
 
   return type;
@@ -4234,20 +4366,46 @@ convert_for_arg_passing (tree type, tree val)
     val = build1 (ADDR_EXPR, build_reference_type (type), val);
   else if (PROMOTE_PROTOTYPES
           && INTEGRAL_TYPE_P (type)
-          && TYPE_PRECISION (type) < TYPE_PRECISION (integer_type_node))
-    val = default_conversion (val);
+          && COMPLETE_TYPE_P (type)
+          && INT_CST_LT_UNSIGNED (TYPE_SIZE (type),
+                                  TYPE_SIZE (integer_type_node)))
+    val = perform_integral_promotions (val);
   return val;
 }
 
+/* Returns true iff FN is a function with magic varargs, i.e. ones for
+   which no conversions at all should be done.  This is true for some
+   builtins which don't act like normal functions.  */
+
+static bool
+magic_varargs_p (tree fn)
+{
+  if (DECL_BUILT_IN (fn))
+    switch (DECL_FUNCTION_CODE (fn))
+      {
+      case BUILT_IN_CLASSIFY_TYPE:
+      case BUILT_IN_CONSTANT_P:
+      case BUILT_IN_NEXT_ARG:
+      case BUILT_IN_STDARG_START:
+      case BUILT_IN_VA_START:
+       return true;
+
+      default:;
+      }
+
+  return false;
+}
+
 /* Subroutine of the various build_*_call functions.  Overload resolution
    has chosen a winning candidate CAND; build up a CALL_EXPR accordingly.
    ARGS is a TREE_LIST of the unconverted arguments to the call.  FLAGS is a
    bitmask of various LOOKUP_* flags which apply to the call itself.  */
 
 static tree
-build_over_call (struct z_candidate *cand, tree args, int flags)
+build_over_call (struct z_candidate *cand, int flags)
 {
   tree fn = cand->fn;
+  tree args = cand->args;
   tree convs = cand->convs;
   tree converted_args = NULL_TREE;
   tree parm = TYPE_ARG_TYPES (TREE_TYPE (fn));
@@ -4261,7 +4419,39 @@ build_over_call (struct z_candidate *cand, tree args, int flags)
       joust (cand, WRAPPER_ZC (TREE_VALUE (val)), 1);
 
   if (DECL_FUNCTION_MEMBER_P (fn))
-    enforce_access (cand->access_path, fn);
+    {
+      /* If FN is a template function, two cases must be considered.
+        For example:
+
+          struct A {
+            protected:
+              template <class T> void f();
+          };
+          template <class T> struct B {
+            protected:
+              void g();
+          };
+          struct C : A, B<int> {
+            using A::f;        // #1
+            using B<int>::g;   // #2
+          };
+
+        In case #1 where `A::f' is a member template, DECL_ACCESS is
+        recorded in the primary template but not in its specialization.
+        We check access of FN using its primary template.
+
+        In case #2, where `B<int>::g' has a DECL_TEMPLATE_INFO simply
+        because it is a member of class template B, DECL_ACCESS is
+        recorded in the specialization `B<int>::g'.  We cannot use its
+        primary template because `B<T>::g' and `B<int>::g' may have
+        different access.  */
+      if (DECL_TEMPLATE_INFO (fn)
+         && is_member_template (DECL_TI_TEMPLATE (fn)))
+       perform_or_defer_access_check (cand->access_path,
+                                      DECL_TI_TEMPLATE (fn));
+      else
+       perform_or_defer_access_check (cand->access_path, fn);
+    }
 
   if (args && TREE_CODE (args) != TREE_LIST)
     args = build_tree_list (NULL_TREE, args);
@@ -4310,12 +4500,17 @@ build_over_call (struct z_candidate *cand, tree args, int flags)
                                       TREE_VALUE (arg),
                                       cand->conversion_path,
                                       1);
+      /* Check that the base class is accessible.  */
+      if (!accessible_base_p (TREE_TYPE (argtype), 
+                             BINFO_TYPE (cand->conversion_path)))
+       error ("`%T' is not an accessible base of `%T'",
+              BINFO_TYPE (cand->conversion_path),
+              TREE_TYPE (argtype));
       /* If fn was found by a using declaration, the conversion path
          will be to the derived class, not the base declaring fn. We
          must convert from derived to base.  */
       base_binfo = lookup_base (TREE_TYPE (TREE_TYPE (converted_arg)),
                                TREE_TYPE (parmtype), ba_ignore, NULL);
-      
       converted_arg = build_base_path (PLUS_EXPR, converted_arg,
                                       base_binfo, 1);
       
@@ -4350,10 +4545,14 @@ build_over_call (struct z_candidate *cand, tree args, int flags)
 
   /* Ellipsis */
   for (; arg; arg = TREE_CHAIN (arg))
-    converted_args 
-      = tree_cons (NULL_TREE,
-                  convert_arg_to_ellipsis (TREE_VALUE (arg)),
-                  converted_args);
+    {
+      tree a = TREE_VALUE (arg);
+      if (magic_varargs_p (fn))
+       /* Do no conversions for magic varargs.  */;
+      else
+       a = convert_arg_to_ellipsis (a);
+      converted_args = tree_cons (NULL_TREE, a, converted_args);
+    }
 
   converted_args = nreverse (converted_args);
 
@@ -4432,9 +4631,30 @@ build_over_call (struct z_candidate *cand, tree args, int flags)
     {
       tree to = stabilize_reference
        (build_indirect_ref (TREE_VALUE (converted_args), 0));
+      tree type = TREE_TYPE (to);
+      tree as_base = CLASSTYPE_AS_BASE (type);
 
       arg = build_indirect_ref (TREE_VALUE (TREE_CHAIN (converted_args)), 0);
-      val = build (MODIFY_EXPR, TREE_TYPE (to), to, arg);
+      if (tree_int_cst_equal (TYPE_SIZE (type), TYPE_SIZE (as_base)))
+       val = build (MODIFY_EXPR, TREE_TYPE (to), to, arg);
+      else
+       {
+         /* We must only copy the non-tail padding parts. Use
+            CLASSTYPE_AS_BASE for the bitwise copy.  */
+         tree to_as_base, arg_as_base, base_ptr_type;
+
+         to = save_expr (to);
+         base_ptr_type = build_pointer_type (as_base);
+         to_as_base = build_indirect_ref
+           (build_nop (base_ptr_type, build_unary_op (ADDR_EXPR, to, 0)), 0);
+         arg_as_base = build_indirect_ref
+           (build_nop (base_ptr_type, build_unary_op (ADDR_EXPR, arg, 0)), 0);
+         
+         val = build (MODIFY_EXPR, as_base, to_as_base, arg_as_base);
+         val = build (COMPOUND_EXPR, type, convert_to_void (val, NULL), to);
+         TREE_USED (val) = 1;
+       }
+      
       return val;
     }
 
@@ -4463,10 +4683,22 @@ build_over_call (struct z_candidate *cand, tree args, int flags)
   else
     fn = build_addr_func (fn);
 
+  return build_cxx_call (fn, args, converted_args);
+}
+
+/* Build and return a call to FN, using the the CONVERTED_ARGS.  ARGS
+   gives the original form of the arguments.  This function performs
+   no overload resolution, conversion, or other high-level
+   operations.  */
+
+tree
+build_cxx_call(tree fn, tree args, tree converted_args)
+{
+  tree fndecl;
+
   /* Recognize certain built-in functions so we can make tree-codes
      other than CALL_EXPR.  We do this when it enables fold-const.c
      to do something useful.  */
-
   if (TREE_CODE (fn) == ADDR_EXPR
       && TREE_CODE (TREE_OPERAND (fn, 0)) == FUNCTION_DECL
       && DECL_BUILT_IN (TREE_OPERAND (fn, 0)))
@@ -4477,14 +4709,26 @@ build_over_call (struct z_candidate *cand, tree args, int flags)
        return exp;
     }
 
-  /* Some built-in function calls will be evaluated at
-     compile-time in fold ().  */
-  fn = fold (build_call (fn, converted_args));
+  fn = build_call (fn, converted_args);
+
+  /* If this call might throw an exception, note that fact.  */
+  fndecl = get_callee_fndecl (fn);
+  if ((!fndecl || !TREE_NOTHROW (fndecl)) 
+      && at_function_scope_p ()
+      && cfun)
+    cp_function_chain->can_throw = 1;
+
+  /* Some built-in function calls will be evaluated at compile-time in
+     fold ().  */
+  fn = fold (fn);
+
   if (VOID_TYPE_P (TREE_TYPE (fn)))
     return fn;
+
   fn = require_complete_type (fn);
   if (fn == error_mark_node)
     return error_mark_node;
+
   if (IS_AGGR_TYPE (TREE_TYPE (fn)))
     fn = build_cplus_new (TREE_TYPE (fn), fn);
   return convert_from_reference (fn);
@@ -4523,7 +4767,7 @@ build_java_interface_fn_ref (tree fn, tree instance)
 
   /* Get the java.lang.Class pointer for the interface being called.  */
   iface = DECL_CONTEXT (fn);
-  iface_ref = lookup_field (iface, get_identifier ("class$"), 0, 0);
+  iface_ref = lookup_field (iface, get_identifier ("class$"), 0, false);
   if (!iface_ref || TREE_CODE (iface_ref) != VAR_DECL
       || DECL_CONTEXT (iface_ref) != iface)
     {
@@ -4614,11 +4858,23 @@ build_special_member_call (tree instance, tree name, tree args,
       TREE_TYPE (instance) = build_pointer_type (class_type);
       instance = build1 (INDIRECT_REF, class_type, instance);
     }
-  else if (name == complete_dtor_identifier 
-          || name == base_dtor_identifier
-          || name == deleting_dtor_identifier)
-    my_friendly_assert (args == NULL_TREE, 20020712);
+  else
+    {
+      if (name == complete_dtor_identifier 
+         || name == base_dtor_identifier
+         || name == deleting_dtor_identifier)
+       my_friendly_assert (args == NULL_TREE, 20020712);
 
+      /* We must perform the conversion here so that we do not
+        subsequently check to see whether BINFO is an accessible
+        base.  (It is OK for a constructor to call a constructor in
+        an inaccessible base as long as the constructor being called
+        is accessible.)  */
+      if (!same_type_ignoring_top_level_qualifiers_p 
+         (TREE_TYPE (instance), BINFO_TYPE (binfo)))
+       instance = convert_to_base_statically (instance, binfo);
+    }
+  
   my_friendly_assert (instance != NULL_TREE, 20020712);
 
   /* Resolve the name.  */
@@ -4647,8 +4903,6 @@ build_special_member_call (tree instance, tree name, tree args,
                          current_in_charge_parm, integer_zero_node),
                   current_vtt_parm,
                   vtt);
-      if (TREE_VIA_VIRTUAL (binfo))
-       binfo = binfo_for_vbase (class_type, current_class_type);
       my_friendly_assert (BINFO_SUBVTT_INDEX (binfo), 20010110);
       sub_vtt = build (PLUS_EXPR, TREE_TYPE (vtt), vtt,
                       BINFO_SUBVTT_INDEX (binfo));
@@ -4656,7 +4910,9 @@ build_special_member_call (tree instance, tree name, tree args,
       args = tree_cons (NULL_TREE, sub_vtt, args);
     }
 
-  return build_new_method_call (instance, fns, args, binfo, flags);
+  return build_new_method_call (instance, fns, args, 
+                               TYPE_BINFO (BINFO_TYPE (binfo)), 
+                               flags);
 }
 
 /* Return the NAME, as a C string.  The NAME indicates a function that
@@ -4709,11 +4965,14 @@ build_new_method_call (tree instance, tree fns, tree args,
   tree mem_args = NULL_TREE, instance_ptr;
   tree name;
   tree user_args;
-  tree templates = NULL_TREE;
   tree call;
   tree fn;
   tree class_type;
   int template_only = 0;
+  bool any_viable_p;
+  tree orig_instance;
+  tree orig_fns;
+  tree orig_args;
 
   my_friendly_assert (instance != NULL_TREE, 20020729);
 
@@ -4722,14 +4981,26 @@ build_new_method_call (tree instance, tree fns, tree args,
       || args == error_mark_node)
     return error_mark_node;
 
+  orig_instance = instance;
+  orig_fns = fns;
+  orig_args = args;
+
+  if (processing_template_decl)
+    {
+      instance = build_non_dependent_expr (instance);
+      if (!BASELINK_P (fns)
+         && TREE_CODE (fns) != PSEUDO_DTOR_EXPR
+         && TREE_TYPE (fns) != unknown_type_node)
+       fns = build_non_dependent_expr (fns);
+      args = build_non_dependent_args (orig_args);
+    }
+
   /* Process the argument list.  */
   user_args = args;
   args = resolve_args (args);
   if (args == error_mark_node)
     return error_mark_node;
 
-  if (TREE_CODE (instance) == OFFSET_REF)
-    instance = resolve_offset_ref (instance);
   if (TREE_CODE (TREE_TYPE (instance)) == REFERENCE_TYPE)
     instance = convert_from_reference (instance);
   basetype = TYPE_MAIN_VARIANT (TREE_TYPE (instance));
@@ -4739,7 +5010,7 @@ build_new_method_call (tree instance, tree fns, tree args,
     {
       call = build_field_call (instance_ptr, fns, args);
       if (call)
-       return call;
+       goto finish;
       error ("call to non-function `%D'", fns);
       return error_mark_node;
     }
@@ -4790,7 +5061,7 @@ build_new_method_call (tree instance, tree fns, tree args,
     {
       tree type = build_pointer_type (basetype);
       if (!same_type_p (type, TREE_TYPE (instance_ptr)))
-       instance_ptr = build1 (NOP_EXPR, type, instance_ptr);
+       instance_ptr = build_nop (type, instance_ptr);
     }
 
   class_type = (conversion_path ? BINFO_TYPE (conversion_path) : NULL_TREE);
@@ -4812,29 +5083,26 @@ build_new_method_call (tree instance, tree fns, tree args,
        this_arglist = args;
 
       if (TREE_CODE (t) == TEMPLATE_DECL)
-       {
-         /* A member template.  */
-         templates = tree_cons (NULL_TREE, t, templates);
-         candidates = 
-           add_template_candidate (candidates, t, 
-                                   class_type,
-                                   explicit_targs,
-                                   this_arglist, optype,
-                                   access_binfo, 
-                                   conversion_path,
-                                   flags,
-                                   DEDUCE_CALL);
-       }
+       /* A member template.  */
+       add_template_candidate (&candidates, t, 
+                               class_type,
+                               explicit_targs,
+                               this_arglist, optype,
+                               access_binfo, 
+                               conversion_path,
+                               flags,
+                               DEDUCE_CALL);
       else if (! template_only)
-       candidates = add_function_candidate (candidates, t, 
-                                            class_type,
-                                            this_arglist,
-                                            access_binfo,
-                                            conversion_path,
-                                            flags);
+       add_function_candidate (&candidates, t, 
+                               class_type,
+                               this_arglist,
+                               access_binfo,
+                               conversion_path,
+                               flags);
     }
 
-  if (! any_viable (candidates))
+  candidates = splice_viable (candidates, pedantic, &any_viable_p);
+  if (!any_viable_p)
     {
       /* XXX will LOOKUP_SPECULATIVELY be needed when this is done?  */
       if (flags & LOOKUP_SPECULATIVELY)
@@ -4856,9 +5124,8 @@ build_new_method_call (tree instance, tree fns, tree args,
       print_z_candidates (candidates);
       return error_mark_node;
     }
-  candidates = splice_viable (candidates);
-  cand = tourney (candidates);
 
+  cand = tourney (candidates);
   if (cand == 0)
     {
       char *pretty_name;
@@ -4895,16 +5162,22 @@ build_new_method_call (tree instance, tree fns, tree args,
     flags |= LOOKUP_NONVIRTUAL;
 
   if (TREE_CODE (TREE_TYPE (cand->fn)) == METHOD_TYPE)
-    call = build_over_call (cand, mem_args, flags);
+    call = build_over_call (cand, flags);
   else
     {
-      call = build_over_call (cand, args, flags);
+      call = build_over_call (cand, flags);
       /* In an expression of the form `a->f()' where `f' turns out to
         be a static member function, `a' is none-the-less evaluated.  */
       if (!is_dummy_object (instance_ptr) && TREE_SIDE_EFFECTS (instance))
        call = build (COMPOUND_EXPR, TREE_TYPE (call), instance, call);
     }
-
+ finish:;
+  
+  if (processing_template_decl && call != error_mark_node)
+    return build_min_non_dep
+      (CALL_EXPR, call,
+       build_min_nt (COMPONENT_REF, orig_instance, orig_fns),
+       orig_args);
   return call;
 }
 
@@ -4985,7 +5258,7 @@ maybe_handle_implicit_object (tree *ics)
       tree reference_type;
 
       /* The `this' parameter is a pointer to a class type.  Make the
-        implict conversion talk about a reference to that same class
+        implicit conversion talk about a reference to that same class
         type.  */
       reference_type = TREE_TYPE (TREE_TYPE (*ics));
       reference_type = build_reference_type (reference_type);
@@ -5078,7 +5351,7 @@ compare_ics (tree ics1, tree ics2)
     {
       /* XXX Isn't this an extension? */
       /* Both ICS are bad.  We try to make a decision based on what
-        would have happenned if they'd been good.  */
+        would have happened if they'd been good.  */
       if (ICS_USER_FLAG (ics1) > ICS_USER_FLAG (ics2)
          || ICS_STD_RANK (ics1) > ICS_STD_RANK (ics2))
        return -1;
@@ -5198,25 +5471,17 @@ compare_ics (tree ics1, tree ics2)
      for pointers A*, except opposite: if B is derived from A then
      A::* converts to B::*, not vice versa.  For that reason, we
      switch the from_ and to_ variables here.  */
-  else if (TYPE_PTRMEM_P (from_type1)
-          && TYPE_PTRMEM_P (from_type2)
-          && TYPE_PTRMEM_P (to_type1)
-          && TYPE_PTRMEM_P (to_type2))
-    {
-      deref_to_type1 = TYPE_OFFSET_BASETYPE (TREE_TYPE (from_type1));
-      deref_to_type2 = TYPE_OFFSET_BASETYPE (TREE_TYPE (from_type2));
-      deref_from_type1 = TYPE_OFFSET_BASETYPE (TREE_TYPE (to_type1));
-      deref_from_type2 = TYPE_OFFSET_BASETYPE (TREE_TYPE (to_type2));
-    }
-  else if (TYPE_PTRMEMFUNC_P (from_type1)
-          && TYPE_PTRMEMFUNC_P (from_type2)
-          && TYPE_PTRMEMFUNC_P (to_type1)
-          && TYPE_PTRMEMFUNC_P (to_type2))
+  else if ((TYPE_PTRMEM_P (from_type1) && TYPE_PTRMEM_P (from_type2)
+           && TYPE_PTRMEM_P (to_type1) && TYPE_PTRMEM_P (to_type2))
+          || (TYPE_PTRMEMFUNC_P (from_type1)
+              && TYPE_PTRMEMFUNC_P (from_type2)
+              && TYPE_PTRMEMFUNC_P (to_type1)
+              && TYPE_PTRMEMFUNC_P (to_type2)))
     {
-      deref_to_type1 = TYPE_PTRMEMFUNC_OBJECT_TYPE (from_type1);
-      deref_to_type2 = TYPE_PTRMEMFUNC_OBJECT_TYPE (from_type2);
-      deref_from_type1 = TYPE_PTRMEMFUNC_OBJECT_TYPE (to_type1);
-      deref_from_type2 = TYPE_PTRMEMFUNC_OBJECT_TYPE (to_type2);
+      deref_to_type1 = TYPE_PTRMEM_CLASS_TYPE (from_type1);
+      deref_to_type2 = TYPE_PTRMEM_CLASS_TYPE (from_type2);
+      deref_from_type1 = TYPE_PTRMEM_CLASS_TYPE (to_type1);
+      deref_from_type2 = TYPE_PTRMEM_CLASS_TYPE (to_type2);
     }
 
   if (deref_from_type1 != NULL_TREE
@@ -5388,19 +5653,6 @@ add_warning (struct z_candidate *winner, struct z_candidate *loser)
                                winner->warnings);
 }
 
-/* Returns true iff functions are equivalent. Equivalent functions are
-   not '==' only if one is a function-local extern function or if
-   both are extern "C".  */
-
-static inline int
-equal_functions (tree fn1, tree fn2)
-{
-  if (DECL_LOCAL_FUNCTION_P (fn1) || DECL_LOCAL_FUNCTION_P (fn2)
-      || DECL_EXTERN_C_FUNCTION_P (fn1))
-    return decls_match (fn1, fn2);
-  return fn1 == fn2;
-}
-
 /* Compare two candidates for overloading as described in
    [over.match.best].  Return values:
 
@@ -5507,45 +5759,52 @@ joust (struct z_candidate *cand1, struct z_candidate *cand2, bool warn)
   /* warn about confusing overload resolution for user-defined conversions,
      either between a constructor and a conversion op, or between two
      conversion ops.  */
-  if (winner && cand1->second_conv
-      && ((DECL_CONSTRUCTOR_P (cand1->fn)
-          != DECL_CONSTRUCTOR_P (cand2->fn))
-         /* Don't warn if the two conv ops convert to the same type...  */
-         || (! DECL_CONSTRUCTOR_P (cand1->fn)
-             && ! same_type_p (TREE_TYPE (TREE_TYPE (cand1->fn)),
-                               TREE_TYPE (TREE_TYPE (cand2->fn))))))
-    {
-      int comp = compare_ics (cand1->second_conv, cand2->second_conv);
-      if (comp != winner)
+  if (winner && warn_conversion && cand1->second_conv
+      && (!DECL_CONSTRUCTOR_P (cand1->fn) || !DECL_CONSTRUCTOR_P (cand2->fn))
+      && winner != compare_ics (cand1->second_conv, cand2->second_conv))
+    {
+      struct z_candidate *w, *l;
+      bool give_warning = false;
+      
+      if (winner == 1)
+       w = cand1, l = cand2;
+      else
+       w = cand2, l = cand1;
+      
+      /* We don't want to complain about `X::operator T1 ()'
+        beating `X::operator T2 () const', when T2 is a no less
+        cv-qualified version of T1.  */
+      if (DECL_CONTEXT (w->fn) == DECL_CONTEXT (l->fn)
+         && !DECL_CONSTRUCTOR_P (w->fn) && !DECL_CONSTRUCTOR_P (l->fn))
        {
-         struct z_candidate *w, *l;
-         tree convn;
-         if (winner == 1)
-           w = cand1, l = cand2;
-         else
-           w = cand2, l = cand1;
-         if (DECL_CONTEXT (cand1->fn) == DECL_CONTEXT (cand2->fn)
-             && ! DECL_CONSTRUCTOR_P (cand1->fn)
-             && ! DECL_CONSTRUCTOR_P (cand2->fn)
-             && (convn = standard_conversion
-                 (TREE_TYPE (TREE_TYPE (l->fn)),
-                  TREE_TYPE (TREE_TYPE (w->fn)), NULL_TREE))
-             && TREE_CODE (convn) == QUAL_CONV)
-           /* Don't complain about `operator char *()' beating
-              `operator const char *() const'.  */;
-         else if (warn)
+         tree t = TREE_TYPE (TREE_TYPE (l->fn));
+         tree f = TREE_TYPE (TREE_TYPE (w->fn));
+         
+         if (TREE_CODE (t) == TREE_CODE (f) && POINTER_TYPE_P (t))
            {
-             tree source = source_type (TREE_VEC_ELT (w->convs, 0));
-             if (! DECL_CONSTRUCTOR_P (w->fn))
-               source = TREE_TYPE (source);
-             warning ("choosing `%D' over `%D'", w->fn, l->fn);
-             warning ("  for conversion from `%T' to `%T'",
-                         source, TREE_TYPE (w->second_conv));
-             warning ("  because conversion sequence for the argument is better");
+             t = TREE_TYPE (t);
+             f = TREE_TYPE (f);
            }
-         else
-           add_warning (w, l);
+         if (!comp_ptr_ttypes (t, f))
+           give_warning = true;
+       }
+      else
+       give_warning = true;
+      
+      if (!give_warning)
+       /*NOP*/;
+      else if (warn)
+       {
+         tree source = source_type (TREE_VEC_ELT (w->convs, 0));
+         if (! DECL_CONSTRUCTOR_P (w->fn))
+           source = TREE_TYPE (source);
+         warning ("choosing `%D' over `%D'", w->fn, l->fn);
+         warning ("  for conversion from `%T' to `%T'",
+                  source, TREE_TYPE (w->second_conv));
+         warning ("  because conversion sequence for the argument is better");
        }
+      else
+       add_warning (w, l);
     }
 
   if (winner)
@@ -5581,7 +5840,6 @@ joust (struct z_candidate *cand1, struct z_candidate *cand2, bool warn)
          TREE_VEC_LENGTH (cand1->convs)
         - (DECL_NONSTATIC_MEMBER_FUNCTION_P (cand1->fn)
            - DECL_CONSTRUCTOR_P (cand1->fn)));
-      /* HERE */
       if (winner)
         return winner;
     }
@@ -5638,7 +5896,7 @@ joust (struct z_candidate *cand1, struct z_candidate *cand2, bool warn)
   if (DECL_P (cand1->fn) && DECL_P (cand2->fn)
       && equal_functions (cand1->fn, cand2->fn))
     return 1;
-
 tweak:
 
   /* Extension: If the worst conversion for one candidate is worse than the
@@ -5663,9 +5921,12 @@ tweak:
         {
          if (warn)
            {
-             pedwarn ("choosing `%D' over `%D'", w->fn, l->fn);
-             pedwarn (
-"  because worst conversion for the former is better than worst conversion for the latter");
+             pedwarn ("\
+ISO C++ says that these are ambiguous, even \
+though the worst conversion for the first is better than \
+the worst conversion for the second:");
+             print_z_candidate (_("candidate 1:"), w);
+             print_z_candidate (_("candidate 2:"), l);
            }
          else
            add_warning (w, l);
@@ -5781,22 +6042,226 @@ perform_implicit_conversion (tree type, tree expr)
   return convert_like (conv, expr);
 }
 
+/* Convert EXPR to TYPE (as a direct-initialization) if that is
+   permitted.  If the conversion is valid, the converted expression is
+   returned.  Otherwise, NULL_TREE is returned, except in the case
+   that TYPE is a class type; in that case, an error is issued.  */
+
+tree
+perform_direct_initialization_if_possible (tree type, tree expr)
+{
+  tree conv;
+  
+  if (type == error_mark_node || error_operand_p (expr))
+    return error_mark_node;
+  /* [dcl.init]
+
+     If the destination type is a (possibly cv-qualified) class type:
+
+     -- If the initialization is direct-initialization ...,
+     constructors are considered. ... If no constructor applies, or
+     the overload resolution is ambiguous, the initialization is
+     ill-formed.  */
+  if (CLASS_TYPE_P (type))
+    {
+      expr = build_special_member_call (NULL_TREE, complete_ctor_identifier,
+                                       build_tree_list (NULL_TREE, expr),
+                                       TYPE_BINFO (type),
+                                       LOOKUP_NORMAL);
+      return build_cplus_new (type, expr);
+    }
+  conv = implicit_conversion (type, TREE_TYPE (expr), expr,
+                             LOOKUP_NORMAL);
+  if (!conv || ICS_BAD_FLAG (conv))
+    return NULL_TREE;
+  return convert_like_real (conv, expr, NULL_TREE, 0, 0, 
+                           /*issue_conversion_warnings=*/false);
+}
+
+/* DECL is a VAR_DECL whose type is a REFERENCE_TYPE.  The reference
+   is being bound to a temporary.  Create and return a new VAR_DECL
+   with the indicated TYPE; this variable will store the value to
+   which the reference is bound.  */
+
+tree 
+make_temporary_var_for_ref_to_temp (tree decl, tree type)
+{
+  tree var;
+
+  /* Create the variable.  */
+  var = build_decl (VAR_DECL, NULL_TREE, type);
+  DECL_ARTIFICIAL (var) = 1;
+  TREE_USED (var) = 1;
+
+  /* Register the variable.  */
+  if (TREE_STATIC (decl))
+    {
+      /* Namespace-scope or local static; give it a mangled name.  */
+      tree name;
+
+      TREE_STATIC (var) = 1;
+      name = mangle_ref_init_variable (decl);
+      DECL_NAME (var) = name;
+      SET_DECL_ASSEMBLER_NAME (var, name);
+      var = pushdecl_top_level (var);
+    }
+  else
+    {
+      /* Create a new cleanup level if necessary.  */
+      maybe_push_cleanup_level (type);
+      /* Don't push unnamed temps.  Do set DECL_CONTEXT, though.  */
+      DECL_CONTEXT (var) = current_function_decl;
+    }
+
+  return var;
+}
+
 /* Convert EXPR to the indicated reference TYPE, in a way suitable for
-   initializing a variable of that TYPE.  Return the converted
-   expression.  */
+   initializing a variable of that TYPE.  If DECL is non-NULL, it is
+   the VAR_DECL being initialized with the EXPR.  (In that case, the
+   type of DECL will be TYPE.)  If DECL is non-NULL, then CLEANUP must
+   also be non-NULL, and with *CLEANUP initialized to NULL.  Upon
+   return, if *CLEANUP is no longer NULL, it will be a CLEANUP_STMT
+   that should be inserted after the returned expression is used to
+   initialize DECL.
+
+   Return the converted expression.  */
 
 tree
-initialize_reference (tree type, tree expr)
+initialize_reference (tree type, tree expr, tree decl, tree *cleanup)
 {
   tree conv;
 
+  if (type == error_mark_node || error_operand_p (expr))
+    return error_mark_node;
+
   conv = reference_binding (type, TREE_TYPE (expr), expr, LOOKUP_NORMAL);
   if (!conv || ICS_BAD_FLAG (conv))
     {
-      error ("could not convert `%E' to `%T'", expr, type);
+      if (!(TYPE_QUALS (TREE_TYPE (type)) & TYPE_QUAL_CONST)
+          && !real_lvalue_p (expr))
+        error ("invalid initialization of non-const reference of "
+               "type '%T' from a temporary of type '%T'",
+               type, TREE_TYPE (expr));
+      else
+        error ("invalid initialization of reference of type "
+              "'%T' from expression of type '%T'", type, 
+              TREE_TYPE (expr));
       return error_mark_node;
     }
 
+  /* If DECL is non-NULL, then this special rule applies:
+
+       [class.temporary]
+
+       The temporary to which the reference is bound or the temporary
+       that is the complete object to which the reference is bound
+       persists for the lifetime of the reference.
+
+       The temporaries created during the evaluation of the expression
+       initializing the reference, except the temporary to which the
+       reference is bound, are destroyed at the end of the
+       full-expression in which they are created.
+
+     In that case, we store the converted expression into a new
+     VAR_DECL in a new scope.  
+
+     However, we want to be careful not to create temporaries when
+     they are not required.  For example, given:
+
+       struct B {}; 
+       struct D : public B {};
+       D f();
+       const B& b = f();
+
+     there is no need to copy the return value from "f"; we can just
+     extend its lifetime.  Similarly, given:
+
+       struct S {};
+       struct T { operator S(); };
+       T t;
+       const S& s = t;
+
+    we can extend the lifetime of the return value of the conversion
+    operator.  */
+  my_friendly_assert (TREE_CODE (conv) == REF_BIND, 20030302);
+  if (decl)
+    {
+      tree var;
+      tree base_conv_type;
+
+      /* Skip over the REF_BIND.  */
+      conv = TREE_OPERAND (conv, 0);
+      /* If the next conversion is a BASE_CONV, skip that too -- but
+        remember that the conversion was required.  */
+      if (TREE_CODE (conv) == BASE_CONV && !NEED_TEMPORARY_P (conv))
+       {
+         base_conv_type = TREE_TYPE (conv);
+         conv = TREE_OPERAND (conv, 0);
+       }
+      else
+       base_conv_type = NULL_TREE;
+      /* Perform the remainder of the conversion.  */
+      expr = convert_like (conv, expr);
+      if (!real_lvalue_p (expr))
+       {
+         tree init;
+         tree type;
+
+         /* Create the temporary variable.  */
+         type = TREE_TYPE (expr);
+         var = make_temporary_var_for_ref_to_temp (decl, type);
+         layout_decl (var, 0);
+         /* Create the INIT_EXPR that will initialize the temporary
+            variable.  */
+         init = build (INIT_EXPR, type, var, expr);
+         if (at_function_scope_p ())
+           {
+             add_decl_stmt (var);
+             *cleanup = cxx_maybe_build_cleanup (var);
+             if (*cleanup)
+               /* We must be careful to destroy the temporary only
+                  after its initialization has taken place.  If the
+                  initialization throws an exception, then the
+                  destructor should not be run.  We cannot simply
+                  transform INIT into something like:
+            
+                    (INIT, ({ CLEANUP_STMT; }))
+
+                  because emit_local_var always treats the
+                  initializer as a full-expression.  Thus, the
+                  destructor would run too early; it would run at the
+                  end of initializing the reference variable, rather
+                  than at the end of the block enclosing the
+                  reference variable.
+
+                  The solution is to pass back a CLEANUP_STMT which
+                  the caller is responsible for attaching to the
+                  statement tree.  */
+               *cleanup = build_stmt (CLEANUP_STMT, var, *cleanup);
+           }
+         else
+           {
+             rest_of_decl_compilation (var, NULL, /*toplev=*/1, at_eof);
+             if (TYPE_HAS_NONTRIVIAL_DESTRUCTOR (type))
+               static_aggregates = tree_cons (NULL_TREE, var,
+                                              static_aggregates);
+           }
+         /* Use its address to initialize the reference variable.  */
+         expr = build_address (var);
+         expr = build (COMPOUND_EXPR, TREE_TYPE (expr), init, expr);
+       }
+      else
+       /* Take the address of EXPR.  */
+       expr = build_unary_op (ADDR_EXPR, expr, 0);
+      /* If a BASE_CONV was required, perform it now.  */
+      if (base_conv_type)
+       expr = (perform_implicit_conversion 
+               (build_pointer_type (base_conv_type), expr));
+      return build_nop (type, expr);
+    }
+
+  /* Perform the conversion.  */
   return convert_like (conv, expr);
 }
 
This page took 0.135775 seconds and 5 git commands to generate.