OSDN Git Service

compiler: Don't crash for invalid constant types for && or ||.
[pf3gnuchains/gcc-fork.git] / gcc / go / gofrontend / expressions.cc
index e308b9a..f9c80b3 100644 (file)
@@ -6,6 +6,8 @@
 
 #include "go-system.h"
 
+#include <algorithm>
+
 #include <gmp.h>
 
 #ifndef ENABLE_BUILD_WITH_CXX
@@ -41,7 +43,7 @@ extern "C"
 // Class Expression.
 
 Expression::Expression(Expression_classification classification,
-                      source_location location)
+                      Location location)
   : classification_(classification), location_(location)
 {
 }
@@ -50,57 +52,6 @@ Expression::~Expression()
 {
 }
 
-// If this expression has a constant integer value, return it.
-
-bool
-Expression::integer_constant_value(bool iota_is_constant, mpz_t val,
-                                  Type** ptype) const
-{
-  *ptype = NULL;
-  return this->do_integer_constant_value(iota_is_constant, val, ptype);
-}
-
-// If this expression has a constant floating point value, return it.
-
-bool
-Expression::float_constant_value(mpfr_t val, Type** ptype) const
-{
-  *ptype = NULL;
-  if (this->do_float_constant_value(val, ptype))
-    return true;
-  mpz_t ival;
-  mpz_init(ival);
-  Type* t;
-  bool ret;
-  if (!this->do_integer_constant_value(false, ival, &t))
-    ret = false;
-  else
-    {
-      mpfr_set_z(val, ival, GMP_RNDN);
-      ret = true;
-    }
-  mpz_clear(ival);
-  return ret;
-}
-
-// If this expression has a constant complex value, return it.
-
-bool
-Expression::complex_constant_value(mpfr_t real, mpfr_t imag,
-                                  Type** ptype) const
-{
-  *ptype = NULL;
-  if (this->do_complex_constant_value(real, imag, ptype))
-    return true;
-  Type *t;
-  if (this->float_constant_value(real, &t))
-    {
-      mpfr_set_ui(imag, 0, GMP_RNDN);
-      return true;
-    }
-  return false;
-}
-
 // Traverse the expressions.
 
 int
@@ -203,11 +154,8 @@ Expression::determine_type_no_context()
 tree
 Expression::convert_for_assignment(Translate_context* context, Type* lhs_type,
                                   Type* rhs_type, tree rhs_tree,
-                                  source_location location)
+                                  Location location)
 {
-  if (lhs_type == rhs_type)
-    return rhs_tree;
-
   if (lhs_type->is_error() || rhs_type->is_error())
     return error_mark_node;
 
@@ -220,7 +168,7 @@ Expression::convert_for_assignment(Translate_context* context, Type* lhs_type,
   if (lhs_type_tree == error_mark_node)
     return error_mark_node;
 
-  if (lhs_type->interface_type() != NULL)
+  if (lhs_type != rhs_type && lhs_type->interface_type() != NULL)
     {
       if (rhs_type->interface_type() == NULL)
        return Expression::convert_type_to_interface(context, lhs_type,
@@ -231,7 +179,7 @@ Expression::convert_for_assignment(Translate_context* context, Type* lhs_type,
                                                          rhs_type, rhs_tree,
                                                          false, location);
     }
-  else if (rhs_type->interface_type() != NULL)
+  else if (lhs_type != rhs_type && rhs_type->interface_type() != NULL)
     return Expression::convert_interface_to_type(context, lhs_type, rhs_type,
                                                 rhs_tree, location);
   else if (lhs_type->is_slice_type() && rhs_type->is_nil_type())
@@ -283,16 +231,24 @@ Expression::convert_for_assignment(Translate_context* context, Type* lhs_type,
           || INTEGRAL_TYPE_P(lhs_type_tree)
           || SCALAR_FLOAT_TYPE_P(lhs_type_tree)
           || COMPLEX_FLOAT_TYPE_P(lhs_type_tree))
-    return fold_convert_loc(location, lhs_type_tree, rhs_tree);
-  else if (TREE_CODE(lhs_type_tree) == RECORD_TYPE
-          && TREE_CODE(TREE_TYPE(rhs_tree)) == RECORD_TYPE)
-    {
+    return fold_convert_loc(location.gcc_location(), lhs_type_tree, rhs_tree);
+  else if ((TREE_CODE(lhs_type_tree) == RECORD_TYPE
+           && TREE_CODE(TREE_TYPE(rhs_tree)) == RECORD_TYPE)
+          || (TREE_CODE(lhs_type_tree) == ARRAY_TYPE
+              && TREE_CODE(TREE_TYPE(rhs_tree)) == ARRAY_TYPE))
+    {
+      // Avoid confusion from zero sized variables which may be
+      // represented as non-zero-sized.
+      if (int_size_in_bytes(lhs_type_tree) == 0
+         || int_size_in_bytes(TREE_TYPE(rhs_tree)) == 0)
+       return rhs_tree;
+
       // This conversion must be permitted by Go, or we wouldn't have
       // gotten here.
       go_assert(int_size_in_bytes(lhs_type_tree)
-                == int_size_in_bytes(TREE_TYPE(rhs_tree)));
-      return fold_build1_loc(location, VIEW_CONVERT_EXPR, lhs_type_tree,
-                            rhs_tree);
+               == int_size_in_bytes(TREE_TYPE(rhs_tree)));
+      return fold_build1_loc(location.gcc_location(), VIEW_CONVERT_EXPR,
+                             lhs_type_tree, rhs_tree);
     }
   else
     {
@@ -307,7 +263,7 @@ Expression::convert_for_assignment(Translate_context* context, Type* lhs_type,
 tree
 Expression::convert_type_to_interface(Translate_context* context,
                                      Type* lhs_type, Type* rhs_type,
-                                     tree rhs_tree, source_location location)
+                                     tree rhs_tree, Location location)
 {
   Gogo* gogo = context->gogo();
   Interface_type* lhs_interface_type = lhs_type->interface_type();
@@ -356,8 +312,8 @@ Expression::convert_type_to_interface(Translate_context* context,
        method_table =
          rhs_named_type->interface_method_table(gogo, lhs_interface_type,
                                                 is_pointer);
-      first_field_value = fold_convert_loc(location, const_ptr_type_node,
-                                          method_table);
+      first_field_value = fold_convert_loc(location.gcc_location(),
+                                           const_ptr_type_node, method_table);
     }
   if (first_field_value == error_mark_node)
     return error_mark_node;
@@ -371,7 +327,8 @@ Expression::convert_type_to_interface(Translate_context* context,
   go_assert(strcmp(IDENTIFIER_POINTER(DECL_NAME(field)),
                    (lhs_is_empty ? "__type_descriptor" : "__methods")) == 0);
   elt->index = field;
-  elt->value = fold_convert_loc(location, TREE_TYPE(field), first_field_value);
+  elt->value = fold_convert_loc(location.gcc_location(), TREE_TYPE(field),
+                                first_field_value);
 
   elt = VEC_quick_push(constructor_elt, init, NULL);
   field = DECL_CHAIN(field);
@@ -392,16 +349,17 @@ Expression::convert_type_to_interface(Translate_context* context,
   tree object_size = TYPE_SIZE_UNIT(TREE_TYPE(rhs_tree));
 
   tree space = gogo->allocate_memory(rhs_type, object_size, location);
-  space = fold_convert_loc(location, build_pointer_type(TREE_TYPE(rhs_tree)),
-                          space);
+  space = fold_convert_loc(location.gcc_location(),
+                           build_pointer_type(TREE_TYPE(rhs_tree)), space);
   space = save_expr(space);
 
-  tree ref = build_fold_indirect_ref_loc(location, space);
+  tree ref = build_fold_indirect_ref_loc(location.gcc_location(), space);
   TREE_THIS_NOTRAP(ref) = 1;
-  tree set = fold_build2_loc(location, MODIFY_EXPR, void_type_node,
-                            ref, rhs_tree);
+  tree set = fold_build2_loc(location.gcc_location(), MODIFY_EXPR,
+                             void_type_node, ref, rhs_tree);
 
-  elt->value = fold_convert_loc(location, TREE_TYPE(field), space);
+  elt->value = fold_convert_loc(location.gcc_location(), TREE_TYPE(field),
+                                space);
 
   return build2(COMPOUND_EXPR, lhs_type_tree, set,
                build_constructor(lhs_type_tree, init));
@@ -414,7 +372,7 @@ Expression::convert_type_to_interface(Translate_context* context,
 tree
 Expression::get_interface_type_descriptor(Translate_context*,
                                          Type* rhs_type, tree rhs_tree,
-                                         source_location location)
+                                         Location location)
 {
   tree rhs_type_tree = TREE_TYPE(rhs_tree);
   go_assert(TREE_CODE(rhs_type_tree) == RECORD_TYPE);
@@ -432,18 +390,20 @@ Expression::get_interface_type_descriptor(Translate_context*,
             == 0);
   go_assert(POINTER_TYPE_P(TREE_TYPE(v)));
   v = save_expr(v);
-  tree v1 = build_fold_indirect_ref_loc(location, v);
+  tree v1 = build_fold_indirect_ref_loc(location.gcc_location(), v);
   go_assert(TREE_CODE(TREE_TYPE(v1)) == RECORD_TYPE);
   tree f = TYPE_FIELDS(TREE_TYPE(v1));
   go_assert(strcmp(IDENTIFIER_POINTER(DECL_NAME(f)), "__type_descriptor")
             == 0);
   v1 = build3(COMPONENT_REF, TREE_TYPE(f), v1, f, NULL_TREE);
 
-  tree eq = fold_build2_loc(location, EQ_EXPR, boolean_type_node, v,
-                           fold_convert_loc(location, TREE_TYPE(v),
-                                            null_pointer_node));
-  tree n = fold_convert_loc(location, TREE_TYPE(v1), null_pointer_node);
-  return fold_build3_loc(location, COND_EXPR, TREE_TYPE(v1),
+  tree eq = fold_build2_loc(location.gcc_location(), EQ_EXPR, boolean_type_node,
+                            v, fold_convert_loc(location.gcc_location(),
+                                                TREE_TYPE(v),
+                                                null_pointer_node));
+  tree n = fold_convert_loc(location.gcc_location(), TREE_TYPE(v1),
+                            null_pointer_node);
+  return fold_build3_loc(location.gcc_location(), COND_EXPR, TREE_TYPE(v1),
                         eq, n, v1);
 }
 
@@ -454,7 +414,7 @@ tree
 Expression::convert_interface_to_interface(Translate_context* context,
                                           Type *lhs_type, Type *rhs_type,
                                           tree rhs_tree, bool for_type_guard,
-                                          source_location location)
+                                          Location location)
 {
   Gogo* gogo = context->gogo();
   Interface_type* lhs_interface_type = lhs_type->interface_type();
@@ -509,7 +469,8 @@ Expression::convert_interface_to_interface(Translate_context* context,
        return error_mark_node;
       // This will panic if the interface conversion fails.
       TREE_NOTHROW(assert_interface_decl) = 0;
-      elt->value = fold_convert_loc(location, TREE_TYPE(field), call);
+      elt->value = fold_convert_loc(location.gcc_location(), TREE_TYPE(field),
+                                    call);
     }
   else if (lhs_is_empty)
     {
@@ -517,8 +478,8 @@ Expression::convert_interface_to_interface(Translate_context* context,
       // first field is just the type descriptor of the object.
       go_assert(strcmp(IDENTIFIER_POINTER(DECL_NAME(field)),
                        "__type_descriptor") == 0);
-      go_assert(TREE_TYPE(field) == TREE_TYPE(rhs_type_descriptor));
-      elt->value = rhs_type_descriptor;
+      elt->value = fold_convert_loc(location.gcc_location(),
+                                   TREE_TYPE(field), rhs_type_descriptor);
     }
   else
     {
@@ -542,7 +503,8 @@ Expression::convert_interface_to_interface(Translate_context* context,
        return error_mark_node;
       // This will panic if the interface conversion fails.
       TREE_NOTHROW(convert_interface_decl) = 0;
-      elt->value = fold_convert_loc(location, TREE_TYPE(field), call);
+      elt->value = fold_convert_loc(location.gcc_location(), TREE_TYPE(field),
+                                    call);
     }
 
   // The second field is simply the object pointer.
@@ -568,7 +530,7 @@ Expression::convert_interface_to_interface(Translate_context* context,
 tree
 Expression::convert_interface_to_type(Translate_context* context,
                                      Type *lhs_type, Type* rhs_type,
-                                     tree rhs_tree, source_location location)
+                                     tree rhs_tree, Location location)
 {
   Gogo* gogo = context->gogo();
   tree rhs_type_tree = TREE_TYPE(rhs_tree);
@@ -621,12 +583,13 @@ Expression::convert_interface_to_type(Translate_context* context,
   // Otherwise it points to the value.
   if (lhs_type->points_to() == NULL)
     {
-      val = fold_convert_loc(location, build_pointer_type(lhs_type_tree), val);
-      val = build_fold_indirect_ref_loc(location, val);
+      val = fold_convert_loc(location.gcc_location(),
+                             build_pointer_type(lhs_type_tree), val);
+      val = build_fold_indirect_ref_loc(location.gcc_location(), val);
     }
 
   return build2(COMPOUND_EXPR, lhs_type_tree, call,
-               fold_convert_loc(location, lhs_type_tree, val));
+               fold_convert_loc(location.gcc_location(), lhs_type_tree, val));
 }
 
 // Convert an expression to a tree.  This is implemented by the child
@@ -748,14 +711,14 @@ Expression::complex_constant_tree(mpfr_t real, mpfr_t imag, tree type)
 
 tree
 Expression::check_bounds(tree val, tree bound_type, tree sofar,
-                        source_location loc)
+                        Location loc)
 {
   tree val_type = TREE_TYPE(val);
   tree ret = NULL_TREE;
 
   if (!TYPE_UNSIGNED(val_type))
     {
-      ret = fold_build2_loc(loc, LT_EXPR, boolean_type_node, val,
+      ret = fold_build2_loc(loc.gcc_location(), LT_EXPR, boolean_type_node, val,
                            build_int_cst(val_type, 0));
       if (ret == boolean_false_node)
        ret = NULL_TREE;
@@ -770,15 +733,16 @@ Expression::check_bounds(tree val, tree bound_type, tree sofar,
          && !TYPE_UNSIGNED(bound_type)))
     {
       tree max = TYPE_MAX_VALUE(bound_type);
-      tree big = fold_build2_loc(loc, GT_EXPR, boolean_type_node, val,
-                                fold_convert_loc(loc, val_type, max));
+      tree big = fold_build2_loc(loc.gcc_location(), GT_EXPR, boolean_type_node,
+                                 val, fold_convert_loc(loc.gcc_location(),
+                                                       val_type, max));
       if (big == boolean_false_node)
        ;
       else if (ret == NULL_TREE)
        ret = big;
       else
-       ret = fold_build2_loc(loc, TRUTH_OR_EXPR, boolean_type_node,
-                             ret, big);
+       ret = fold_build2_loc(loc.gcc_location(), TRUTH_OR_EXPR,
+                              boolean_type_node, ret, big);
     }
 
   if (ret == NULL_TREE)
@@ -786,7 +750,7 @@ Expression::check_bounds(tree val, tree bound_type, tree sofar,
   else if (sofar == NULL_TREE)
     return ret;
   else
-    return fold_build2_loc(loc, TRUTH_OR_EXPR, boolean_type_node,
+    return fold_build2_loc(loc.gcc_location(), TRUTH_OR_EXPR, boolean_type_node,
                           sofar, ret);
 }
 
@@ -801,7 +765,7 @@ Expression::dump_expression(Ast_dump_context* ast_dump_context) const
 class Error_expression : public Expression
 {
  public:
-  Error_expression(source_location location)
+  Error_expression(Location location)
     : Expression(EXPRESSION_ERROR, location)
   { }
 
@@ -811,24 +775,9 @@ class Error_expression : public Expression
   { return true; }
 
   bool
-  do_integer_constant_value(bool, mpz_t val, Type**) const
-  {
-    mpz_set_ui(val, 0);
-    return true;
-  }
-
-  bool
-  do_float_constant_value(mpfr_t val, Type**) const
+  do_numeric_constant_value(Numeric_constant* nc) const
   {
-    mpfr_set_ui(val, 0, GMP_RNDN);
-    return true;
-  }
-
-  bool
-  do_complex_constant_value(mpfr_t real, mpfr_t imag, Type**) const
-  {
-    mpfr_set_ui(real, 0, GMP_RNDN);
-    mpfr_set_ui(imag, 0, GMP_RNDN);
+    nc->set_unsigned_long(NULL, 0);
     return true;
   }
 
@@ -869,7 +818,7 @@ Error_expression::do_dump_expression(Ast_dump_context* ast_dump_context) const
 }
 
 Expression*
-Expression::make_error(source_location location)
+Expression::make_error(Location location)
 {
   return new Error_expression(location);
 }
@@ -881,7 +830,7 @@ class
 Type_expression : public Expression
 {
  public:
-  Type_expression(Type* type, source_location location)
+  Type_expression(Type* type, Location location)
     : Expression(EXPRESSION_TYPE, location),
       type_(type)
   { }
@@ -925,7 +874,7 @@ Type_expression::do_dump_expression(Ast_dump_context* ast_dump_context) const
 }
 
 Expression*
-Expression::make_type(Type* type, source_location location)
+Expression::make_type(Type* type, Location location)
 {
   return new Type_expression(type, location);
 }
@@ -1037,7 +986,7 @@ Var_expression::do_get_tree(Translate_context* context)
     go_unreachable();
   if (is_in_heap)
     {
-      ret = build_fold_indirect_ref_loc(this->location(), ret);
+      ret = build_fold_indirect_ref_loc(this->location().gcc_location(), ret);
       TREE_THIS_NOTRAP(ret) = 1;
     }
   return ret;
@@ -1054,7 +1003,7 @@ Var_expression::do_dump_expression(Ast_dump_context* ast_dump_context) const
 // Make a reference to a variable in an expression.
 
 Expression*
-Expression::make_var_reference(Named_object* var, source_location location)
+Expression::make_var_reference(Named_object* var, Location location)
 {
   if (var->is_sink())
     return Expression::make_sink(location);
@@ -1104,7 +1053,7 @@ Temporary_reference_expression::do_get_tree(Translate_context* context)
     {
       Btype* type_btype = this->type()->base()->get_backend(context->gogo());
       tree type_tree = type_to_tree(type_btype);
-      ret = fold_convert_loc(this->location(), type_tree, ret);
+      ret = fold_convert_loc(this->location().gcc_location(), type_tree, ret);
     }
   return ret;
 }
@@ -1122,17 +1071,74 @@ Temporary_reference_expression::do_dump_expression(
 
 Temporary_reference_expression*
 Expression::make_temporary_reference(Temporary_statement* statement,
-                                    source_location location)
+                                    Location location)
 {
   return new Temporary_reference_expression(statement, location);
 }
 
+// Class Set_and_use_temporary_expression.
+
+// Return the type.
+
+Type*
+Set_and_use_temporary_expression::do_type()
+{
+  return this->statement_->type();
+}
+
+// Take the address.
+
+void
+Set_and_use_temporary_expression::do_address_taken(bool)
+{
+  this->statement_->set_is_address_taken();
+}
+
+// Return the backend representation.
+
+tree
+Set_and_use_temporary_expression::do_get_tree(Translate_context* context)
+{
+  Bvariable* bvar = this->statement_->get_backend_variable(context);
+  tree var_tree = var_to_tree(bvar);
+  tree expr_tree = this->expr_->get_tree(context);
+  if (var_tree == error_mark_node || expr_tree == error_mark_node)
+    return error_mark_node;
+  Location loc = this->location();
+  return build2_loc(loc.gcc_location(), COMPOUND_EXPR, TREE_TYPE(var_tree),
+                   build2_loc(loc.gcc_location(), MODIFY_EXPR, void_type_node,
+                              var_tree, expr_tree),
+                   var_tree);
+}
+
+// Dump.
+
+void
+Set_and_use_temporary_expression::do_dump_expression(
+    Ast_dump_context* ast_dump_context) const
+{
+  ast_dump_context->ostream() << '(';
+  ast_dump_context->dump_temp_variable_name(this->statement_);
+  ast_dump_context->ostream() << " = ";
+  this->expr_->dump_expression(ast_dump_context);
+  ast_dump_context->ostream() << ')';
+}
+
+// Make a set-and-use temporary.
+
+Set_and_use_temporary_expression*
+Expression::make_set_and_use_temporary(Temporary_statement* statement,
+                                      Expression* expr, Location location)
+{
+  return new Set_and_use_temporary_expression(statement, expr, location);
+}
+
 // A sink expression--a use of the blank identifier _.
 
 class Sink_expression : public Expression
 {
  public:
-  Sink_expression(source_location location)
+  Sink_expression(Location location)
     : Expression(EXPRESSION_SINK, location),
       type_(NULL), var_(NULL_TREE)
   { }
@@ -1210,7 +1216,7 @@ Sink_expression::do_dump_expression(Ast_dump_context* ast_dump_context) const
 // Make a sink expression.
 
 Expression*
-Expression::make_sink(source_location location)
+Expression::make_sink(Location location)
 {
   return new Sink_expression(location);
 }
@@ -1263,7 +1269,8 @@ Func_expression::get_tree_without_closure(Gogo* gogo)
   // can't take their address.
   if (fntype->is_builtin())
     {
-      error_at(this->location(), "invalid use of special builtin function %qs",
+      error_at(this->location(),
+              "invalid use of special builtin function %qs; must be called",
               this->function_->name().c_str());
       return error_mark_node;
     }
@@ -1285,7 +1292,7 @@ Func_expression::get_tree_without_closure(Gogo* gogo)
   if (fndecl == error_mark_node)
     return error_mark_node;
 
-  return build_fold_addr_expr_loc(this->location(), fndecl);
+  return build_fold_addr_expr_loc(this->location().gcc_location(), fndecl);
 }
 
 // Get the tree for a function expression.  This is used when we take
@@ -1305,30 +1312,18 @@ Func_expression::do_get_tree(Translate_context* context)
             && TREE_CODE(TREE_OPERAND(fnaddr, 0)) == FUNCTION_DECL);
   TREE_ADDRESSABLE(TREE_OPERAND(fnaddr, 0)) = 1;
 
-  // For a normal non-nested function call, that is all we have to do.
-  if (!this->function_->is_function()
-      || this->function_->func_value()->enclosing() == NULL)
-    {
-      go_assert(this->closure_ == NULL);
-      return fnaddr;
-    }
+  // If there is no closure, that is all have to do.
+  if (this->closure_ == NULL)
+    return fnaddr;
 
-  // For a nested function call, we have to always allocate a
-  // trampoline.  If we don't always allocate, then closures will not
-  // be reliably distinct.
-  Expression* closure = this->closure_;
-  tree closure_tree;
-  if (closure == NULL)
-    closure_tree = null_pointer_node;
-  else
-    {
-      // Get the value of the closure.  This will be a pointer to
-      // space allocated on the heap.
-      closure_tree = closure->get_tree(context);
-      if (closure_tree == error_mark_node)
-       return error_mark_node;
-      go_assert(POINTER_TYPE_P(TREE_TYPE(closure_tree)));
-    }
+  go_assert(this->function_->func_value()->enclosing() != NULL);
+
+  // Get the value of the closure.  This will be a pointer to space
+  // allocated on the heap.
+  tree closure_tree = this->closure_->get_tree(context);
+  if (closure_tree == error_mark_node)
+    return error_mark_node;
+  go_assert(POINTER_TYPE_P(TREE_TYPE(closure_tree)));
 
   // Now we need to build some code on the heap.  This code will load
   // the static chain pointer with the closure and then jump to the
@@ -1357,7 +1352,7 @@ Func_expression::do_dump_expression(Ast_dump_context* ast_dump_context) const
 
 Expression*
 Expression::make_func_reference(Named_object* function, Expression* closure,
-                               source_location location)
+                               Location location)
 {
   return new Func_expression(function, closure, location);
 }
@@ -1377,7 +1372,7 @@ Unknown_expression::name() const
 Expression*
 Unknown_expression::do_lower(Gogo*, Named_object*, Statement_inserter*, int)
 {
-  source_location location = this->location();
+  Location location = this->location();
   Named_object* no = this->named_object_;
   Named_object* real;
   if (!no->is_unknown())
@@ -1389,8 +1384,9 @@ Unknown_expression::do_lower(Gogo*, Named_object*, Statement_inserter*, int)
        {
          if (this->is_composite_literal_key_)
            return this;
-         error_at(location, "reference to undefined name %qs",
-                  this->named_object_->message_name().c_str());
+         if (!this->no_error_message_)
+           error_at(location, "reference to undefined name %qs",
+                    this->named_object_->message_name().c_str());
          return Expression::make_error(location);
        }
     }
@@ -1403,10 +1399,12 @@ Unknown_expression::do_lower(Gogo*, Named_object*, Statement_inserter*, int)
     case Named_object::NAMED_OBJECT_TYPE_DECLARATION:
       if (this->is_composite_literal_key_)
        return this;
-      error_at(location, "reference to undefined type %qs",
-              real->message_name().c_str());
+      if (!this->no_error_message_)
+       error_at(location, "reference to undefined type %qs",
+                real->message_name().c_str());
       return Expression::make_error(location);
     case Named_object::NAMED_OBJECT_VAR:
+      real->var_value()->set_is_used();
       return Expression::make_var_reference(real, location);
     case Named_object::NAMED_OBJECT_FUNC:
     case Named_object::NAMED_OBJECT_FUNC_DECLARATION:
@@ -1414,7 +1412,8 @@ Unknown_expression::do_lower(Gogo*, Named_object*, Statement_inserter*, int)
     case Named_object::NAMED_OBJECT_PACKAGE:
       if (this->is_composite_literal_key_)
        return this;
-      error_at(location, "unexpected reference to package");
+      if (!this->no_error_message_)
+       error_at(location, "unexpected reference to package");
       return Expression::make_error(location);
     default:
       go_unreachable();
@@ -1432,8 +1431,8 @@ Unknown_expression::do_dump_expression(Ast_dump_context* ast_dump_context) const
 
 // Make a reference to an unknown name.
 
-Expression*
-Expression::make_unknown_reference(Named_object* no, source_location location)
+Unknown_expression*
+Expression::make_unknown_reference(Named_object* no, Location location)
 {
   return new Unknown_expression(no, location);
 }
@@ -1443,7 +1442,7 @@ Expression::make_unknown_reference(Named_object* no, source_location location)
 class Boolean_expression : public Expression
 {
  public:
-  Boolean_expression(bool val, source_location location)
+  Boolean_expression(bool val, Location location)
     : Expression(EXPRESSION_BOOLEAN, location),
       val_(val), type_(NULL)
   { }
@@ -1528,7 +1527,7 @@ Boolean_expression::do_import(Import* imp)
 // Make a boolean expression.
 
 Expression*
-Expression::make_boolean(bool val, source_location location)
+Expression::make_boolean(bool val, Location location)
 {
   return new Boolean_expression(val, location);
 }
@@ -1665,7 +1664,7 @@ String_expression::do_dump_expression(Ast_dump_context* ast_dump_context) const
 // Make a string expression.
 
 Expression*
-Expression::make_string(const std::string& val, source_location location)
+Expression::make_string(const std::string& val, Location location)
 {
   return new String_expression(val, location);
 }
@@ -1675,18 +1674,15 @@ Expression::make_string(const std::string& val, source_location location)
 class Integer_expression : public Expression
 {
  public:
-  Integer_expression(const mpz_t* val, Type* type, source_location location)
+  Integer_expression(const mpz_t* val, Type* type, bool is_character_constant,
+                    Location location)
     : Expression(EXPRESSION_INTEGER, location),
-      type_(type)
+      type_(type), is_character_constant_(is_character_constant)
   { mpz_init_set(this->val_, *val); }
 
   static Expression*
   do_import(Import*);
 
-  // Return whether VAL fits in the type.
-  static bool
-  check_constant(mpz_t val, Type*, source_location);
-
   // Write VAL to string dump.
   static void
   export_integer(String_dump* exp, const mpz_t val);
@@ -1701,7 +1697,7 @@ class Integer_expression : public Expression
   { return true; }
 
   bool
-  do_integer_constant_value(bool, mpz_t val, Type** ptype) const;
+  do_numeric_constant_value(Numeric_constant* nc) const;
 
   Type*
   do_type();
@@ -1717,8 +1713,14 @@ class Integer_expression : public Expression
 
   Expression*
   do_copy()
-  { return Expression::make_integer(&this->val_, this->type_,
-                                   this->location()); }
+  {
+    if (this->is_character_constant_)
+      return Expression::make_character(&this->val_, this->type_,
+                                       this->location());
+    else
+      return Expression::make_integer(&this->val_, this->type_,
+                                     this->location());
+  }
 
   void
   do_export(Export*) const;
@@ -1731,17 +1733,20 @@ class Integer_expression : public Expression
   mpz_t val_;
   // The type so far.
   Type* type_;
+  // Whether this is a character constant.
+  bool is_character_constant_;
 };
 
-// Return an integer constant value.
+// Return a numeric constant for this expression.  We have to mark
+// this as a character when appropriate.
 
 bool
-Integer_expression::do_integer_constant_value(bool, mpz_t val,
-                                             Type** ptype) const
+Integer_expression::do_numeric_constant_value(Numeric_constant* nc) const
 {
-  if (this->type_ != NULL)
-    *ptype = this->type_;
-  mpz_set(val, this->val_);
+  if (this->is_character_constant_)
+    nc->set_rune(this->type_, this->val_);
+  else
+    nc->set_int(this->type_, this->val_);
   return true;
 }
 
@@ -1752,7 +1757,12 @@ Type*
 Integer_expression::do_type()
 {
   if (this->type_ == NULL)
-    this->type_ = Type::make_abstract_integer_type();
+    {
+      if (this->is_character_constant_)
+       this->type_ = Type::make_abstract_character_type();
+      else
+       this->type_ = Type::make_abstract_integer_type();
+    }
   return this->type_;
 }
 
@@ -1764,53 +1774,15 @@ Integer_expression::do_determine_type(const Type_context* context)
 {
   if (this->type_ != NULL && !this->type_->is_abstract())
     ;
-  else if (context->type != NULL
-          && (context->type->integer_type() != NULL
-              || context->type->float_type() != NULL
-              || context->type->complex_type() != NULL))
+  else if (context->type != NULL && context->type->is_numeric_type())
     this->type_ = context->type;
   else if (!context->may_be_abstract)
-    this->type_ = Type::lookup_integer_type("int");
-}
-
-// Return true if the integer VAL fits in the range of the type TYPE.
-// Otherwise give an error and return false.  TYPE may be NULL.
-
-bool
-Integer_expression::check_constant(mpz_t val, Type* type,
-                                  source_location location)
-{
-  if (type == NULL)
-    return true;
-  Integer_type* itype = type->integer_type();
-  if (itype == NULL || itype->is_abstract())
-    return true;
-
-  int bits = mpz_sizeinbase(val, 2);
-
-  if (itype->is_unsigned())
-    {
-      // For an unsigned type we can only accept a nonnegative number,
-      // and we must be able to represent at least BITS.
-      if (mpz_sgn(val) >= 0
-         && bits <= itype->bits())
-       return true;
-    }
-  else
     {
-      // For a signed type we need an extra bit to indicate the sign.
-      // We have to handle the most negative integer specially.
-      if (bits + 1 <= itype->bits()
-         || (bits <= itype->bits()
-             && mpz_sgn(val) < 0
-             && (mpz_scan1(val, 0)
-                 == static_cast<unsigned long>(itype->bits() - 1))
-             && mpz_scan0(val, itype->bits()) == ULONG_MAX))
-       return true;
+      if (this->is_character_constant_)
+       this->type_ = Type::lookup_integer_type("int32");
+      else
+       this->type_ = Type::lookup_integer_type("int");
     }
-
-  error_at(location, "integer constant overflow");
-  return false;
 }
 
 // Check the type of an integer constant.
@@ -1818,10 +1790,15 @@ Integer_expression::check_constant(mpz_t val, Type* type,
 void
 Integer_expression::do_check_types(Gogo*)
 {
-  if (this->type_ == NULL)
+  Type* type = this->type_;
+  if (type == NULL)
     return;
-  if (!Integer_expression::check_constant(this->val_, this->type_,
-                                         this->location()))
+  Numeric_constant nc;
+  if (this->is_character_constant_)
+    nc.set_rune(NULL, this->val_);
+  else
+    nc.set_int(NULL, this->val_);
+  if (!nc.set_type(type, true, this->location()))
     this->set_is_error();
 }
 
@@ -1885,6 +1862,8 @@ void
 Integer_expression::do_export(Export* exp) const
 {
   Integer_expression::export_integer(exp, this->val_);
+  if (this->is_character_constant_)
+    exp->write_c_string("'");
   // A trailing space lets us reliably identify the end of the number.
   exp->write_c_string(" ");
 }
@@ -1948,6 +1927,10 @@ Integer_expression::do_import(Import* imp)
   else if (num.find('.') == std::string::npos
           && num.find('E') == std::string::npos)
     {
+      bool is_character_constant = (!num.empty()
+                                   && num[num.length() - 1] == '\'');
+      if (is_character_constant)
+       num = num.substr(0, num.length() - 1);
       mpz_t val;
       if (mpz_init_set_str(val, num.c_str(), 10) != 0)
        {
@@ -1955,7 +1938,11 @@ Integer_expression::do_import(Import* imp)
                   num.c_str());
          return Expression::make_error(imp->location());
        }
-      Expression* ret = Expression::make_integer(&val, NULL, imp->location());
+      Expression* ret;
+      if (is_character_constant)
+       ret = Expression::make_character(&val, NULL, imp->location());
+      else
+       ret = Expression::make_integer(&val, NULL, imp->location());
       mpz_clear(val);
       return ret;
     }
@@ -1978,16 +1965,27 @@ Integer_expression::do_import(Import* imp)
 void
 Integer_expression::do_dump_expression(Ast_dump_context* ast_dump_context) const
 {
+  if (this->is_character_constant_)
+    ast_dump_context->ostream() << '\'';
   Integer_expression::export_integer(ast_dump_context, this->val_);
+  if (this->is_character_constant_)
+    ast_dump_context->ostream() << '\'';
 }
 
 // Build a new integer value.
 
 Expression*
-Expression::make_integer(const mpz_t* val, Type* type,
-                        source_location location)
+Expression::make_integer(const mpz_t* val, Type* type, Location location)
+{
+  return new Integer_expression(val, type, false, location);
+}
+
+// Build a new character constant value.
+
+Expression*
+Expression::make_character(const mpz_t* val, Type* type, Location location)
 {
-  return new Integer_expression(val, type, location);
+  return new Integer_expression(val, type, true, location);
 }
 
 // Floats.
@@ -1995,21 +1993,13 @@ Expression::make_integer(const mpz_t* val, Type* type,
 class Float_expression : public Expression
 {
  public:
-  Float_expression(const mpfr_t* val, Type* type, source_location location)
+  Float_expression(const mpfr_t* val, Type* type, Location location)
     : Expression(EXPRESSION_FLOAT, location),
       type_(type)
   {
     mpfr_init_set(this->val_, *val, GMP_RNDN);
   }
 
-  // Constrain VAL to fit into TYPE.
-  static void
-  constrain_float(mpfr_t val, Type* type);
-
-  // Return whether VAL fits in the type.
-  static bool
-  check_constant(mpfr_t val, Type*, source_location);
-
   // Write VAL to export data.
   static void
   export_float(String_dump* exp, const mpfr_t val);
@@ -2024,7 +2014,11 @@ class Float_expression : public Expression
   { return true; }
 
   bool
-  do_float_constant_value(mpfr_t val, Type**) const;
+  do_numeric_constant_value(Numeric_constant* nc) const
+  {
+    nc->set_float(this->type_, this->val_);
+    return true;
+  }
 
   Type*
   do_type();
@@ -2056,27 +2050,6 @@ class Float_expression : public Expression
   Type* type_;
 };
 
-// Constrain VAL to fit into TYPE.
-
-void
-Float_expression::constrain_float(mpfr_t val, Type* type)
-{
-  Float_type* ftype = type->float_type();
-  if (ftype != NULL && !ftype->is_abstract())
-    mpfr_prec_round(val, ftype->bits(), GMP_RNDN);
-}
-
-// Return a floating point constant value.
-
-bool
-Float_expression::do_float_constant_value(mpfr_t val, Type** ptype) const
-{
-  if (this->type_ != NULL)
-    *ptype = this->type_;
-  mpfr_set(val, this->val_, GMP_RNDN);
-  return true;
-}
-
 // Return the current type.  If we haven't set the type yet, we return
 // an abstract float type.
 
@@ -2105,73 +2078,18 @@ Float_expression::do_determine_type(const Type_context* context)
     this->type_ = Type::lookup_float_type("float64");
 }
 
-// Return true if the floating point value VAL fits in the range of
-// the type TYPE.  Otherwise give an error and return false.  TYPE may
-// be NULL.
-
-bool
-Float_expression::check_constant(mpfr_t val, Type* type,
-                                source_location location)
-{
-  if (type == NULL)
-    return true;
-  Float_type* ftype = type->float_type();
-  if (ftype == NULL || ftype->is_abstract())
-    return true;
-
-  // A NaN or Infinity always fits in the range of the type.
-  if (mpfr_nan_p(val) || mpfr_inf_p(val) || mpfr_zero_p(val))
-    return true;
-
-  mp_exp_t exp = mpfr_get_exp(val);
-  mp_exp_t max_exp;
-  switch (ftype->bits())
-    {
-    case 32:
-      max_exp = 128;
-      break;
-    case 64:
-      max_exp = 1024;
-      break;
-    default:
-      go_unreachable();
-    }
-  if (exp > max_exp)
-    {
-      error_at(location, "floating point constant overflow");
-      return false;
-    }
-  return true;
-}
-
 // Check the type of a float value.
 
 void
 Float_expression::do_check_types(Gogo*)
 {
-  if (this->type_ == NULL)
+  Type* type = this->type_;
+  if (type == NULL)
     return;
-
-  if (!Float_expression::check_constant(this->val_, this->type_,
-                                       this->location()))
+  Numeric_constant nc;
+  nc.set_float(NULL, this->val_);
+  if (!nc.set_type(this->type_, true, this->location()))
     this->set_is_error();
-
-  Integer_type* integer_type = this->type_->integer_type();
-  if (integer_type != NULL)
-    {
-      if (!mpfr_integer_p(this->val_))
-       this->report_error(_("floating point constant truncated to integer"));
-      else
-       {
-         go_assert(!integer_type->is_abstract());
-         mpz_t ival;
-         mpz_init(ival);
-         mpfr_get_z(ival, this->val_, GMP_RNDN);
-         Integer_expression::check_constant(ival, integer_type,
-                                            this->location());
-         mpz_clear(ival);
-       }
-    }
 }
 
 // Get a tree for a float constant.
@@ -2237,7 +2155,7 @@ Float_expression::do_dump_expression(Ast_dump_context* ast_dump_context) const
 // Make a float expression.
 
 Expression*
-Expression::make_float(const mpfr_t* val, Type* type, source_location location)
+Expression::make_float(const mpfr_t* val, Type* type, Location location)
 {
   return new Float_expression(val, type, location);
 }
@@ -2248,7 +2166,7 @@ class Complex_expression : public Expression
 {
  public:
   Complex_expression(const mpfr_t* real, const mpfr_t* imag, Type* type,
-                    source_location location)
+                    Location location)
     : Expression(EXPRESSION_COMPLEX, location),
       type_(type)
   {
@@ -2256,14 +2174,6 @@ class Complex_expression : public Expression
     mpfr_init_set(this->imag_, *imag, GMP_RNDN);
   }
 
-  // Constrain REAL/IMAG to fit into TYPE.
-  static void
-  constrain_complex(mpfr_t real, mpfr_t imag, Type* type);
-
-  // Return whether REAL/IMAG fits in the type.
-  static bool
-  check_constant(mpfr_t real, mpfr_t imag, Type*, source_location);
-
   // Write REAL/IMAG to string dump.
   static void
   export_complex(String_dump* exp, const mpfr_t real, const mpfr_t val);
@@ -2279,7 +2189,11 @@ class Complex_expression : public Expression
   { return true; }
 
   bool
-  do_complex_constant_value(mpfr_t real, mpfr_t imag, Type**) const;
+  do_numeric_constant_value(Numeric_constant* nc) const
+  {
+    nc->set_complex(this->type_, this->real_, this->imag_);
+    return true;
+  }
 
   Type*
   do_type();
@@ -2315,32 +2229,6 @@ class Complex_expression : public Expression
   Type* type_;
 };
 
-// Constrain REAL/IMAG to fit into TYPE.
-
-void
-Complex_expression::constrain_complex(mpfr_t real, mpfr_t imag, Type* type)
-{
-  Complex_type* ctype = type->complex_type();
-  if (ctype != NULL && !ctype->is_abstract())
-    {
-      mpfr_prec_round(real, ctype->bits() / 2, GMP_RNDN);
-      mpfr_prec_round(imag, ctype->bits() / 2, GMP_RNDN);
-    }
-}
-
-// Return a complex constant value.
-
-bool
-Complex_expression::do_complex_constant_value(mpfr_t real, mpfr_t imag,
-                                             Type** ptype) const
-{
-  if (this->type_ != NULL)
-    *ptype = this->type_;
-  mpfr_set(real, this->real_, GMP_RNDN);
-  mpfr_set(imag, this->imag_, GMP_RNDN);
-  return true;
-}
-
 // Return the current type.  If we haven't set the type yet, we return
 // an abstract complex type.
 
@@ -2367,65 +2255,17 @@ Complex_expression::do_determine_type(const Type_context* context)
     this->type_ = Type::lookup_complex_type("complex128");
 }
 
-// Return true if the complex value REAL/IMAG fits in the range of the
-// type TYPE.  Otherwise give an error and return false.  TYPE may be
-// NULL.
-
-bool
-Complex_expression::check_constant(mpfr_t real, mpfr_t imag, Type* type,
-                                  source_location location)
-{
-  if (type == NULL)
-    return true;
-  Complex_type* ctype = type->complex_type();
-  if (ctype == NULL || ctype->is_abstract())
-    return true;
-
-  mp_exp_t max_exp;
-  switch (ctype->bits())
-    {
-    case 64:
-      max_exp = 128;
-      break;
-    case 128:
-      max_exp = 1024;
-      break;
-    default:
-      go_unreachable();
-    }
-
-  // A NaN or Infinity always fits in the range of the type.
-  if (!mpfr_nan_p(real) && !mpfr_inf_p(real) && !mpfr_zero_p(real))
-    {
-      if (mpfr_get_exp(real) > max_exp)
-       {
-         error_at(location, "complex real part constant overflow");
-         return false;
-       }
-    }
-
-  if (!mpfr_nan_p(imag) && !mpfr_inf_p(imag) && !mpfr_zero_p(imag))
-    {
-      if (mpfr_get_exp(imag) > max_exp)
-       {
-         error_at(location, "complex imaginary part constant overflow");
-         return false;
-       }
-    }
-
-  return true;
-}
-
 // Check the type of a complex value.
 
 void
 Complex_expression::do_check_types(Gogo*)
 {
-  if (this->type_ == NULL)
+  Type* type = this->type_;
+  if (type == NULL)
     return;
-
-  if (!Complex_expression::check_constant(this->real_, this->imag_,
-                                         this->type_, this->location()))
+  Numeric_constant nc;
+  nc.set_complex(NULL, this->real_, this->imag_);
+  if (!nc.set_type(this->type_, true, this->location()))
     this->set_is_error();
 }
 
@@ -2489,7 +2329,7 @@ Complex_expression::do_dump_expression(Ast_dump_context* ast_dump_context) const
 
 Expression*
 Expression::make_complex(const mpfr_t* real, const mpfr_t* imag, Type* type,
-                        source_location location)
+                        Location location)
 {
   return new Complex_expression(real, imag, type, location);
 }
@@ -2525,7 +2365,7 @@ class Find_named_object : public Traverse
 class Const_expression : public Expression
 {
  public:
-  Const_expression(Named_object* constant, source_location location)
+  Const_expression(Named_object* constant, Location location)
     : Expression(EXPRESSION_CONST_REFERENCE, location),
       constant_(constant), type_(NULL), seen_(false)
   { }
@@ -2550,17 +2390,10 @@ class Const_expression : public Expression
   { return true; }
 
   bool
-  do_integer_constant_value(bool, mpz_t val, Type**) const;
-
-  bool
-  do_float_constant_value(mpfr_t val, Type**) const;
-
-  bool
-  do_complex_constant_value(mpfr_t real, mpfr_t imag, Type**) const;
+  do_numeric_constant_value(Numeric_constant* nc) const;
 
   bool
-  do_string_constant_value(std::string* val) const
-  { return this->constant_->const_value()->expr()->string_constant_value(val); }
+  do_string_constant_value(std::string* val) const;
 
   Type*
   do_type();
@@ -2640,110 +2473,49 @@ Const_expression::do_lower(Gogo* gogo, Named_object*,
   return this;
 }
 
-// Return an integer constant value.
+// Return a numeric constant value.
 
 bool
-Const_expression::do_integer_constant_value(bool iota_is_constant, mpz_t val,
-                                           Type** ptype) const
+Const_expression::do_numeric_constant_value(Numeric_constant* nc) const
 {
   if (this->seen_)
     return false;
 
+  Expression* e = this->constant_->const_value()->expr();
+  
+  this->seen_ = true;
+
+  bool r = e->numeric_constant_value(nc);
+
+  this->seen_ = false;
+
   Type* ctype;
   if (this->type_ != NULL)
     ctype = this->type_;
   else
     ctype = this->constant_->const_value()->type();
-  if (ctype != NULL && ctype->integer_type() == NULL)
-    return false;
-
-  Expression* e = this->constant_->const_value()->expr();
-
-  this->seen_ = true;
-
-  Type* t;
-  bool r = e->integer_constant_value(iota_is_constant, val, &t);
-
-  this->seen_ = false;
-
-  if (r
-      && ctype != NULL
-      && !Integer_expression::check_constant(val, ctype, this->location()))
-    return false;
-
-  *ptype = ctype != NULL ? ctype : t;
-  return r;
-}
-
-// Return a floating point constant value.
-
-bool
-Const_expression::do_float_constant_value(mpfr_t val, Type** ptype) const
-{
-  if (this->seen_)
-    return false;
-
-  Type* ctype;
-  if (this->type_ != NULL)
-    ctype = this->type_;
-  else
-    ctype = this->constant_->const_value()->type();
-  if (ctype != NULL && ctype->float_type() == NULL)
-    return false;
-
-  this->seen_ = true;
-
-  Type* t;
-  bool r = this->constant_->const_value()->expr()->float_constant_value(val,
-                                                                       &t);
-
-  this->seen_ = false;
-
   if (r && ctype != NULL)
     {
-      if (!Float_expression::check_constant(val, ctype, this->location()))
+      if (!nc->set_type(ctype, false, this->location()))
        return false;
-      Float_expression::constrain_float(val, ctype);
     }
-  *ptype = ctype != NULL ? ctype : t;
+
   return r;
 }
 
-// Return a complex constant value.
-
 bool
-Const_expression::do_complex_constant_value(mpfr_t real, mpfr_t imag,
-                                           Type **ptype) const
+Const_expression::do_string_constant_value(std::string* val) const
 {
   if (this->seen_)
     return false;
 
-  Type* ctype;
-  if (this->type_ != NULL)
-    ctype = this->type_;
-  else
-    ctype = this->constant_->const_value()->type();
-  if (ctype != NULL && ctype->complex_type() == NULL)
-    return false;
+  Expression* e = this->constant_->const_value()->expr();
 
   this->seen_ = true;
-
-  Type *t;
-  bool r = this->constant_->const_value()->expr()->complex_constant_value(real,
-                                                                         imag,
-                                                                         &t);
-
+  bool ok = e->string_constant_value(val);
   this->seen_ = false;
 
-  if (r && ctype != NULL)
-    {
-      if (!Complex_expression::check_constant(real, imag, ctype,
-                                             this->location()))
-       return false;
-      Complex_expression::constrain_complex(real, imag, ctype);
-    }
-  *ptype = ctype != NULL ? ctype : t;
-  return r;
+  return ok;
 }
 
 // Return the type of the const reference.
@@ -2794,12 +2566,8 @@ Const_expression::do_determine_type(const Type_context* context)
   if (ctype != NULL && !ctype->is_abstract())
     ;
   else if (context->type != NULL
-          && (context->type->integer_type() != NULL
-              || context->type->float_type() != NULL
-              || context->type->complex_type() != NULL)
-          && (cetype->integer_type() != NULL
-              || cetype->float_type() != NULL
-              || cetype->complex_type() != NULL))
+          && context->type->is_numeric_type()
+          && cetype->is_numeric_type())
     this->type_ = context->type;
   else if (context->type != NULL
           && context->type->is_string_type()
@@ -2861,35 +2629,15 @@ Const_expression::do_check_types(Gogo*)
 
   this->check_for_init_loop();
 
-  if (this->type_ == NULL || this->type_->is_abstract())
-    return;
-
-  // Check for integer overflow.
-  if (this->type_->integer_type() != NULL)
+  // Check that numeric constant fits in type.
+  if (this->type_ != NULL && this->type_->is_numeric_type())
     {
-      mpz_t ival;
-      mpz_init(ival);
-      Type* dummy;
-      if (!this->integer_constant_value(true, ival, &dummy))
+      Numeric_constant nc;
+      if (this->constant_->const_value()->expr()->numeric_constant_value(&nc))
        {
-         mpfr_t fval;
-         mpfr_init(fval);
-         Expression* cexpr = this->constant_->const_value()->expr();
-         if (cexpr->float_constant_value(fval, &dummy))
-           {
-             if (!mpfr_integer_p(fval))
-               this->report_error(_("floating point constant "
-                                    "truncated to integer"));
-             else
-               {
-                 mpfr_get_z(ival, fval, GMP_RNDN);
-                 Integer_expression::check_constant(ival, this->type_,
-                                                    this->location());
-               }
-           }
-         mpfr_clear(fval);
+         if (!nc.set_type(this->type_, true, this->location()))
+           this->set_is_error();
        }
-      mpz_clear(ival);
     }
 }
 
@@ -2913,41 +2661,18 @@ Const_expression::do_get_tree(Translate_context* context)
   // object is an abstract int or float, we try to get the abstract
   // value.  Otherwise we may lose something in the conversion.
   if (this->type_ != NULL
+      && this->type_->is_numeric_type()
       && (this->constant_->const_value()->type() == NULL
          || this->constant_->const_value()->type()->is_abstract()))
     {
       Expression* expr = this->constant_->const_value()->expr();
-      mpz_t ival;
-      mpz_init(ival);
-      Type* t;
-      if (expr->integer_constant_value(true, ival, &t))
-       {
-         tree ret = Expression::integer_constant_tree(ival, type_tree);
-         mpz_clear(ival);
-         return ret;
-       }
-      mpz_clear(ival);
-
-      mpfr_t fval;
-      mpfr_init(fval);
-      if (expr->float_constant_value(fval, &t))
-       {
-         tree ret = Expression::float_constant_tree(fval, type_tree);
-         mpfr_clear(fval);
-         return ret;
-       }
-
-      mpfr_t imag;
-      mpfr_init(imag);
-      if (expr->complex_constant_value(fval, imag, &t))
+      Numeric_constant nc;
+      if (expr->numeric_constant_value(&nc)
+         && nc.set_type(this->type_, false, this->location()))
        {
-         tree ret = Expression::complex_constant_tree(fval, imag, type_tree);
-         mpfr_clear(fval);
-         mpfr_clear(imag);
-         return ret;
+         Expression* e = nc.expression(this->location());
+         return e->get_tree(context);
        }
-      mpfr_clear(imag);
-      mpfr_clear(fval);
     }
 
   tree const_tree = this->constant_->get_tree(gogo, context->function());
@@ -2982,7 +2707,7 @@ Const_expression::do_dump_expression(Ast_dump_context* ast_dump_context) const
 
 Expression*
 Expression::make_const_reference(Named_object* constant,
-                                source_location location)
+                                Location location)
 {
   return new Const_expression(constant, location);
 }
@@ -3028,7 +2753,7 @@ Find_named_object::expression(Expression** pexpr)
 class Nil_expression : public Expression
 {
  public:
-  Nil_expression(source_location location)
+  Nil_expression(Location location)
     : Expression(EXPRESSION_NIL, location)
   { }
 
@@ -3077,7 +2802,7 @@ Nil_expression::do_import(Import* imp)
 // Make a nil expression.
 
 Expression*
-Expression::make_nil(source_location location)
+Expression::make_nil(Location location)
 {
   return new Nil_expression(location);
 }
@@ -3090,7 +2815,7 @@ Expression::make_nil(source_location location)
 class Iota_expression : public Parser_expression
 {
  public:
-  Iota_expression(source_location location)
+  Iota_expression(Location location)
     : Parser_expression(EXPRESSION_IOTA, location)
   { }
 
@@ -3115,7 +2840,7 @@ class Iota_expression : public Parser_expression
 Expression*
 Expression::make_iota()
 {
-  static Iota_expression iota_expression(UNKNOWN_LOCATION);
+  static Iota_expression iota_expression(Linemap::unknown_location());
   return &iota_expression;
 }
 
@@ -3125,7 +2850,7 @@ class Type_conversion_expression : public Expression
 {
  public:
   Type_conversion_expression(Type* type, Expression* expr,
-                            source_location location)
+                            Location location)
     : Expression(EXPRESSION_CONVERSION, location),
       type_(type), expr_(expr), may_convert_function_types_(false)
   { }
@@ -3164,13 +2889,7 @@ class Type_conversion_expression : public Expression
   { return this->expr_->is_constant(); }
 
   bool
-  do_integer_constant_value(bool, mpz_t, Type**) const;
-
-  bool
-  do_float_constant_value(mpfr_t, Type**) const;
-
-  bool
-  do_complex_constant_value(mpfr_t, mpfr_t, Type**) const;
+  do_numeric_constant_value(Numeric_constant*) const;
 
   bool
   do_string_constant_value(std::string*) const;
@@ -3234,92 +2953,27 @@ Type_conversion_expression::do_lower(Gogo*, Named_object*,
 {
   Type* type = this->type_;
   Expression* val = this->expr_;
-  source_location location = this->location();
-
-  if (type->integer_type() != NULL)
-    {
-      mpz_t ival;
-      mpz_init(ival);
-      Type* dummy;
-      if (val->integer_constant_value(false, ival, &dummy))
-       {
-         if (!Integer_expression::check_constant(ival, type, location))
-           mpz_set_ui(ival, 0);
-         Expression* ret = Expression::make_integer(&ival, type, location);
-         mpz_clear(ival);
-         return ret;
-       }
-
-      mpfr_t fval;
-      mpfr_init(fval);
-      if (val->float_constant_value(fval, &dummy))
-       {
-         if (!mpfr_integer_p(fval))
-           {
-             error_at(location,
-                      "floating point constant truncated to integer");
-             return Expression::make_error(location);
-           }
-         mpfr_get_z(ival, fval, GMP_RNDN);
-         if (!Integer_expression::check_constant(ival, type, location))
-           mpz_set_ui(ival, 0);
-         Expression* ret = Expression::make_integer(&ival, type, location);
-         mpfr_clear(fval);
-         mpz_clear(ival);
-         return ret;
-       }
-      mpfr_clear(fval);
-      mpz_clear(ival);
-    }
-
-  if (type->float_type() != NULL)
-    {
-      mpfr_t fval;
-      mpfr_init(fval);
-      Type* dummy;
-      if (val->float_constant_value(fval, &dummy))
-       {
-         if (!Float_expression::check_constant(fval, type, location))
-           mpfr_set_ui(fval, 0, GMP_RNDN);
-         Float_expression::constrain_float(fval, type);
-         Expression *ret = Expression::make_float(&fval, type, location);
-         mpfr_clear(fval);
-         return ret;
-       }
-      mpfr_clear(fval);
-    }
+  Location location = this->location();
 
-  if (type->complex_type() != NULL)
+  if (type->is_numeric_type())
     {
-      mpfr_t real;
-      mpfr_t imag;
-      mpfr_init(real);
-      mpfr_init(imag);
-      Type* dummy;
-      if (val->complex_constant_value(real, imag, &dummy))
+      Numeric_constant nc;
+      if (val->numeric_constant_value(&nc))
        {
-         if (!Complex_expression::check_constant(real, imag, type, location))
-           {
-             mpfr_set_ui(real, 0, GMP_RNDN);
-             mpfr_set_ui(imag, 0, GMP_RNDN);
-           }
-         Complex_expression::constrain_complex(real, imag, type);
-         Expression* ret = Expression::make_complex(&real, &imag, type,
-                                                    location);
-         mpfr_clear(real);
-         mpfr_clear(imag);
-         return ret;
+         if (!nc.set_type(type, true, location))
+           return Expression::make_error(location);
+         return nc.expression(location);
        }
-      mpfr_clear(real);
-      mpfr_clear(imag);
     }
 
-  if (type->is_slice_type() && type->named_type() == NULL)
+  if (type->is_slice_type())
     {
       Type* element_type = type->array_type()->element_type()->forwarded();
-      bool is_byte = element_type == Type::lookup_integer_type("uint8");
-      bool is_int = element_type == Type::lookup_integer_type("int");
-      if (is_byte || is_int)
+      bool is_byte = (element_type->integer_type() != NULL
+                     && element_type->integer_type()->is_byte());
+      bool is_rune = (element_type->integer_type() != NULL
+                     && element_type->integer_type()->is_rune());
+      if (is_byte || is_rune)
        {
          std::string s;
          if (val->string_constant_value(&s))
@@ -3374,118 +3028,17 @@ Type_conversion_expression::do_lower(Gogo*, Named_object*,
   return this;
 }
 
-// Return the constant integer value if there is one.
-
-bool
-Type_conversion_expression::do_integer_constant_value(bool iota_is_constant,
-                                                     mpz_t val,
-                                                     Type** ptype) const
-{
-  if (this->type_->integer_type() == NULL)
-    return false;
-
-  mpz_t ival;
-  mpz_init(ival);
-  Type* dummy;
-  if (this->expr_->integer_constant_value(iota_is_constant, ival, &dummy))
-    {
-      if (!Integer_expression::check_constant(ival, this->type_,
-                                             this->location()))
-       {
-         mpz_clear(ival);
-         return false;
-       }
-      mpz_set(val, ival);
-      mpz_clear(ival);
-      *ptype = this->type_;
-      return true;
-    }
-  mpz_clear(ival);
-
-  mpfr_t fval;
-  mpfr_init(fval);
-  if (this->expr_->float_constant_value(fval, &dummy))
-    {
-      mpfr_get_z(val, fval, GMP_RNDN);
-      mpfr_clear(fval);
-      if (!Integer_expression::check_constant(val, this->type_,
-                                             this->location()))
-       return false;
-      *ptype = this->type_;
-      return true;
-    }
-  mpfr_clear(fval);
-
-  return false;
-}
-
-// Return the constant floating point value if there is one.
+// Return the constant numeric value if there is one.
 
 bool
-Type_conversion_expression::do_float_constant_value(mpfr_t val,
-                                                   Type** ptype) const
+Type_conversion_expression::do_numeric_constant_value(
+    Numeric_constant* nc) const
 {
-  if (this->type_->float_type() == NULL)
+  if (!this->type_->is_numeric_type())
     return false;
-
-  mpfr_t fval;
-  mpfr_init(fval);
-  Type* dummy;
-  if (this->expr_->float_constant_value(fval, &dummy))
-    {
-      if (!Float_expression::check_constant(fval, this->type_,
-                                           this->location()))
-       {
-         mpfr_clear(fval);
-         return false;
-       }
-      mpfr_set(val, fval, GMP_RNDN);
-      mpfr_clear(fval);
-      Float_expression::constrain_float(val, this->type_);
-      *ptype = this->type_;
-      return true;
-    }
-  mpfr_clear(fval);
-
-  return false;
-}
-
-// Return the constant complex value if there is one.
-
-bool
-Type_conversion_expression::do_complex_constant_value(mpfr_t real,
-                                                     mpfr_t imag,
-                                                     Type **ptype) const
-{
-  if (this->type_->complex_type() == NULL)
+  if (!this->expr_->numeric_constant_value(nc))
     return false;
-
-  mpfr_t rval;
-  mpfr_t ival;
-  mpfr_init(rval);
-  mpfr_init(ival);
-  Type* dummy;
-  if (this->expr_->complex_constant_value(rval, ival, &dummy))
-    {
-      if (!Complex_expression::check_constant(rval, ival, this->type_,
-                                             this->location()))
-       {
-         mpfr_clear(rval);
-         mpfr_clear(ival);
-         return false;
-       }
-      mpfr_set(real, rval, GMP_RNDN);
-      mpfr_set(imag, ival, GMP_RNDN);
-      mpfr_clear(rval);
-      mpfr_clear(ival);
-      Complex_expression::constrain_complex(real, imag, this->type_);
-      *ptype = this->type_;
-      return true;
-    }
-  mpfr_clear(rval);
-  mpfr_clear(ival);
-
-  return false;  
+  return nc->set_type(this->type_, false, this->location());
 }
 
 // Return the constant string value if there is one.
@@ -3496,20 +3049,17 @@ Type_conversion_expression::do_string_constant_value(std::string* val) const
   if (this->type_->is_string_type()
       && this->expr_->type()->integer_type() != NULL)
     {
-      mpz_t ival;
-      mpz_init(ival);
-      Type* dummy;
-      if (this->expr_->integer_constant_value(false, ival, &dummy))
+      Numeric_constant nc;
+      if (this->expr_->numeric_constant_value(&nc))
        {
-         unsigned long ulval = mpz_get_ui(ival);
-         if (mpz_cmp_ui(ival, ulval) == 0)
+         unsigned long ival;
+         if (nc.to_unsigned_long(&ival) == Numeric_constant::NC_UL_VALID)
            {
-             Lex::append_char(ulval, true, val, this->location());
-             mpz_clear(ival);
+             val->clear();
+             Lex::append_char(ival, true, val, this->location());
              return true;
            }
        }
-      mpz_clear(ival);
     }
 
   // FIXME: Could handle conversion from const []int here.
@@ -3613,28 +3163,19 @@ Type_conversion_expression::do_get_tree(Translate_context* context)
                               integer_type_node,
                               fold_convert(integer_type_node, expr_tree));
     }
-  else if (type->is_string_type()
-          && (expr_type->array_type() != NULL
-              || (expr_type->points_to() != NULL
-                  && expr_type->points_to()->array_type() != NULL)))
+  else if (type->is_string_type() && expr_type->is_slice_type())
     {
-      Type* t = expr_type;
-      if (t->points_to() != NULL)
-       {
-         t = t->points_to();
-         expr_tree = build_fold_indirect_ref(expr_tree);
-       }
       if (!DECL_P(expr_tree))
        expr_tree = save_expr(expr_tree);
-      Array_type* a = t->array_type();
+      Array_type* a = expr_type->array_type();
       Type* e = a->element_type()->forwarded();
       go_assert(e->integer_type() != NULL);
       tree valptr = fold_convert(const_ptr_type_node,
                                 a->value_pointer_tree(gogo, expr_tree));
       tree len = a->length_tree(gogo, expr_tree);
-      len = fold_convert_loc(this->location(), integer_type_node, len);
-      if (e->integer_type()->is_unsigned()
-         && e->integer_type()->bits() == 8)
+      len = fold_convert_loc(this->location().gcc_location(), integer_type_node,
+                             len);
+      if (e->integer_type()->is_byte())
        {
          static tree byte_array_to_string_fndecl;
          ret = Gogo::call_builtin(&byte_array_to_string_fndecl,
@@ -3649,7 +3190,7 @@ Type_conversion_expression::do_get_tree(Translate_context* context)
        }
       else
        {
-         go_assert(e == Type::lookup_integer_type("int"));
+         go_assert(e->integer_type()->is_rune());
          static tree int_array_to_string_fndecl;
          ret = Gogo::call_builtin(&int_array_to_string_fndecl,
                                   this->location(),
@@ -3666,10 +3207,9 @@ Type_conversion_expression::do_get_tree(Translate_context* context)
     {
       Type* e = type->array_type()->element_type()->forwarded();
       go_assert(e->integer_type() != NULL);
-      if (e->integer_type()->is_unsigned()
-         && e->integer_type()->bits() == 8)
+      if (e->integer_type()->is_byte())
        {
-         static tree string_to_byte_array_fndecl;
+         tree string_to_byte_array_fndecl = NULL_TREE;
          ret = Gogo::call_builtin(&string_to_byte_array_fndecl,
                                   this->location(),
                                   "__go_string_to_byte_array",
@@ -3680,8 +3220,8 @@ Type_conversion_expression::do_get_tree(Translate_context* context)
        }
       else
        {
-         go_assert(e == Type::lookup_integer_type("int"));
-         static tree string_to_int_array_fndecl;
+         go_assert(e->integer_type()->is_rune());
+         tree string_to_int_array_fndecl = NULL_TREE;
          ret = Gogo::call_builtin(&string_to_int_array_fndecl,
                                   this->location(),
                                   "__go_string_to_int_array",
@@ -3702,7 +3242,8 @@ Type_conversion_expression::do_get_tree(Translate_context* context)
   else if (this->may_convert_function_types_
           && type->function_type() != NULL
           && expr_type->function_type() != NULL)
-    ret = fold_convert_loc(this->location(), type_tree, expr_tree);
+    ret = fold_convert_loc(this->location().gcc_location(), type_tree,
+                           expr_tree);
   else
     ret = Expression::convert_for_assignment(context, type, expr_type,
                                             expr_tree, this->location());
@@ -3750,7 +3291,7 @@ Type_conversion_expression::do_dump_expression(
 // Make a type cast expression.
 
 Expression*
-Expression::make_cast(Type* type, Expression* val, source_location location)
+Expression::make_cast(Type* type, Expression* val, Location location)
 {
   if (type->is_error_type() || val->is_error_expression())
     return Expression::make_error(location);
@@ -3763,7 +3304,7 @@ class Unsafe_type_conversion_expression : public Expression
 {
  public:
   Unsafe_type_conversion_expression(Type* type, Expression* expr,
-                                   source_location location)
+                                   Location location)
     : Expression(EXPRESSION_UNSAFE_CONVERSION, location),
       type_(type), expr_(expr)
   { }
@@ -3827,7 +3368,7 @@ Unsafe_type_conversion_expression::do_get_tree(Translate_context* context)
   if (type_tree == error_mark_node || expr_tree == error_mark_node)
     return error_mark_node;
 
-  source_location loc = this->location();
+  Location loc = this->location();
 
   bool use_view_convert = false;
   if (t->is_slice_type())
@@ -3839,10 +3380,6 @@ Unsafe_type_conversion_expression::do_get_tree(Translate_context* context)
     go_assert(et->map_type() != NULL);
   else if (t->channel_type() != NULL)
     go_assert(et->channel_type() != NULL);
-  else if (t->points_to() != NULL && t->points_to()->channel_type() != NULL)
-    go_assert((et->points_to() != NULL
-               && et->points_to()->channel_type() != NULL)
-              || et->is_nil_type());
   else if (t->points_to() != NULL)
     go_assert(et->points_to() != NULL || et->is_nil_type());
   else if (et->is_unsafe_pointer_type())
@@ -3873,9 +3410,10 @@ Unsafe_type_conversion_expression::do_get_tree(Translate_context* context)
     go_unreachable();
 
   if (use_view_convert)
-    return fold_build1_loc(loc, VIEW_CONVERT_EXPR, type_tree, expr_tree);
+    return fold_build1_loc(loc.gcc_location(), VIEW_CONVERT_EXPR, type_tree,
+                           expr_tree);
   else
-    return fold_convert_loc(loc, type_tree, expr_tree);
+    return fold_convert_loc(loc.gcc_location(), type_tree, expr_tree);
 }
 
 // Dump ast representation for an unsafe type conversion expression.
@@ -3894,7 +3432,7 @@ Unsafe_type_conversion_expression::do_dump_expression(
 
 Expression*
 Expression::make_unsafe_cast(Type* type, Expression* expr,
-                            source_location location)
+                            Location location)
 {
   return new Unsafe_type_conversion_expression(type, expr, location);
 }
@@ -3904,7 +3442,7 @@ Expression::make_unsafe_cast(Type* type, Expression* expr,
 class Unary_expression : public Expression
 {
  public:
-  Unary_expression(Operator op, Expression* expr, source_location location)
+  Unary_expression(Operator op, Expression* expr, Location location)
     : Expression(EXPRESSION_UNARY, location),
       op_(op), escapes_(true), create_temp_(false), expr_(expr)
   { }
@@ -3936,22 +3474,11 @@ class Unary_expression : public Expression
     this->create_temp_ = true;
   }
 
-  // Apply unary opcode OP to UVAL, setting VAL.  Return true if this
-  // could be done, false if not.
-  static bool
-  eval_integer(Operator op, Type* utype, mpz_t uval, mpz_t val,
-              source_location);
-
-  // Apply unary opcode OP to UVAL, setting VAL.  Return true if this
-  // could be done, false if not.
-  static bool
-  eval_float(Operator op, mpfr_t uval, mpfr_t val);
-
-  // Apply unary opcode OP to UREAL/UIMAG, setting REAL/IMAG.  Return
-  // true if this could be done, false if not.
+  // Apply unary opcode OP to UNC, setting NC.  Return true if this
+  // could be done, false if not.  Issue errors for overflow.
   static bool
-  eval_complex(Operator op, mpfr_t ureal, mpfr_t uimag, mpfr_t real,
-              mpfr_t imag);
+  eval_constant(Operator op, const Numeric_constant* unc,
+               Location, Numeric_constant* nc);
 
   static Expression*
   do_import(Import*);
@@ -3968,13 +3495,7 @@ class Unary_expression : public Expression
   do_is_constant() const;
 
   bool
-  do_integer_constant_value(bool, mpz_t, Type**) const;
-
-  bool
-  do_float_constant_value(mpfr_t, Type**) const;
-
-  bool
-  do_complex_constant_value(mpfr_t, mpfr_t, Type**) const;
+  do_numeric_constant_value(Numeric_constant*) const;
 
   Type*
   do_type();
@@ -4029,7 +3550,7 @@ class Unary_expression : public Expression
 Expression*
 Unary_expression::do_lower(Gogo*, Named_object*, Statement_inserter*, int)
 {
-  source_location loc = this->location();
+  Location loc = this->location();
   Operator op = this->op_;
   Expression* expr = this->expr_;
 
@@ -4073,62 +3594,14 @@ Unary_expression::do_lower(Gogo*, Named_object*, Statement_inserter*, int)
       return Expression::make_error(this->location());
     }
 
-  if (op == OPERATOR_PLUS || op == OPERATOR_MINUS
-      || op == OPERATOR_NOT || op == OPERATOR_XOR)
+  if (op == OPERATOR_PLUS || op == OPERATOR_MINUS || op == OPERATOR_XOR)
     {
-      Expression* ret = NULL;
-
-      mpz_t eval;
-      mpz_init(eval);
-      Type* etype;
-      if (expr->integer_constant_value(false, eval, &etype))
-       {
-         mpz_t val;
-         mpz_init(val);
-         if (Unary_expression::eval_integer(op, etype, eval, val, loc))
-           ret = Expression::make_integer(&val, etype, loc);
-         mpz_clear(val);
-       }
-      mpz_clear(eval);
-      if (ret != NULL)
-       return ret;
-
-      if (op == OPERATOR_PLUS || op == OPERATOR_MINUS)
+      Numeric_constant nc;
+      if (expr->numeric_constant_value(&nc))
        {
-         mpfr_t fval;
-         mpfr_init(fval);
-         Type* ftype;
-         if (expr->float_constant_value(fval, &ftype))
-           {
-             mpfr_t val;
-             mpfr_init(val);
-             if (Unary_expression::eval_float(op, fval, val))
-               ret = Expression::make_float(&val, ftype, loc);
-             mpfr_clear(val);
-           }
-         if (ret != NULL)
-           {
-             mpfr_clear(fval);
-             return ret;
-           }
-
-         mpfr_t ival;
-         mpfr_init(ival);
-         if (expr->complex_constant_value(fval, ival, &ftype))
-           {
-             mpfr_t real;
-             mpfr_t imag;
-             mpfr_init(real);
-             mpfr_init(imag);
-             if (Unary_expression::eval_complex(op, fval, ival, real, imag))
-               ret = Expression::make_complex(&real, &imag, ftype, loc);
-             mpfr_clear(real);
-             mpfr_clear(imag);
-           }
-         mpfr_clear(ival);
-         mpfr_clear(fval);
-         if (ret != NULL)
-           return ret;
+         Numeric_constant result;
+         if (Unary_expression::eval_constant(op, &nc, loc, &result))
+           return result.expression(loc);
        }
     }
 
@@ -4164,180 +3637,174 @@ Unary_expression::do_is_constant() const
     return this->expr_->is_constant();
 }
 
-// Apply unary opcode OP to UVAL, setting VAL.  UTYPE is the type of
-// UVAL, if known; it may be NULL.  Return true if this could be done,
-// false if not.
+// Apply unary opcode OP to UNC, setting NC.  Return true if this
+// could be done, false if not.  Issue errors for overflow.
 
 bool
-Unary_expression::eval_integer(Operator op, Type* utype, mpz_t uval, mpz_t val,
-                              source_location location)
+Unary_expression::eval_constant(Operator op, const Numeric_constant* unc,
+                               Location location, Numeric_constant* nc)
 {
   switch (op)
     {
     case OPERATOR_PLUS:
-      mpz_set(val, uval);
+      *nc = *unc;
       return true;
+
     case OPERATOR_MINUS:
-      mpz_neg(val, uval);
-      return Integer_expression::check_constant(val, utype, location);
-    case OPERATOR_NOT:
-      mpz_set_ui(val, mpz_cmp_si(uval, 0) == 0 ? 1 : 0);
-      return true;
-    case OPERATOR_XOR:
-      if (utype == NULL
-         || utype->integer_type() == NULL
-         || utype->integer_type()->is_abstract())
-       mpz_com(val, uval);
+      if (unc->is_int() || unc->is_rune())
+       break;
+      else if (unc->is_float())
+       {
+         mpfr_t uval;
+         unc->get_float(&uval);
+         mpfr_t val;
+         mpfr_init(val);
+         mpfr_neg(val, uval, GMP_RNDN);
+         nc->set_float(unc->type(), val);
+         mpfr_clear(uval);
+         mpfr_clear(val);
+         return true;
+       }
+      else if (unc->is_complex())
+       {
+         mpfr_t ureal, uimag;
+         unc->get_complex(&ureal, &uimag);
+         mpfr_t real, imag;
+         mpfr_init(real);
+         mpfr_init(imag);
+         mpfr_neg(real, ureal, GMP_RNDN);
+         mpfr_neg(imag, uimag, GMP_RNDN);
+         nc->set_complex(unc->type(), real, imag);
+         mpfr_clear(ureal);
+         mpfr_clear(uimag);
+         mpfr_clear(real);
+         mpfr_clear(imag);
+         return true;
+       }
       else
-       {
-         // The number of HOST_WIDE_INTs that it takes to represent
-         // UVAL.
-         size_t count = ((mpz_sizeinbase(uval, 2)
-                          + HOST_BITS_PER_WIDE_INT
-                          - 1)
-                         / HOST_BITS_PER_WIDE_INT);
-
-         unsigned HOST_WIDE_INT* phwi = new unsigned HOST_WIDE_INT[count];
-         memset(phwi, 0, count * sizeof(HOST_WIDE_INT));
-
-         size_t ecount;
-         mpz_export(phwi, &ecount, -1, sizeof(HOST_WIDE_INT), 0, 0, uval);
-         go_assert(ecount <= count);
-
-         // Trim down to the number of words required by the type.
-         size_t obits = utype->integer_type()->bits();
-         if (!utype->integer_type()->is_unsigned())
-           ++obits;
-         size_t ocount = ((obits + HOST_BITS_PER_WIDE_INT - 1)
-                          / HOST_BITS_PER_WIDE_INT);
-         go_assert(ocount <= count);
-
-         for (size_t i = 0; i < ocount; ++i)
-           phwi[i] = ~phwi[i];
-
-         size_t clearbits = ocount * HOST_BITS_PER_WIDE_INT - obits;
-         if (clearbits != 0)
-           phwi[ocount - 1] &= (((unsigned HOST_WIDE_INT) (HOST_WIDE_INT) -1)
-                                >> clearbits);
+       go_unreachable();
 
-         mpz_import(val, ocount, -1, sizeof(HOST_WIDE_INT), 0, 0, phwi);
+    case OPERATOR_XOR:
+      break;
 
-         delete[] phwi;
-       }
-      return Integer_expression::check_constant(val, utype, location);
+    case OPERATOR_NOT:
     case OPERATOR_AND:
     case OPERATOR_MULT:
       return false;
+
     default:
       go_unreachable();
     }
-}
 
-// Apply unary opcode OP to UVAL, setting VAL.  Return true if this
-// could be done, false if not.
+  if (!unc->is_int() && !unc->is_rune())
+    return false;
+
+  mpz_t uval;
+  if (unc->is_rune())
+    unc->get_rune(&uval);
+  else
+    unc->get_int(&uval);
+  mpz_t val;
+  mpz_init(val);
 
-bool
-Unary_expression::eval_float(Operator op, mpfr_t uval, mpfr_t val)
-{
   switch (op)
     {
-    case OPERATOR_PLUS:
-      mpfr_set(val, uval, GMP_RNDN);
-      return true;
     case OPERATOR_MINUS:
-      mpfr_neg(val, uval, GMP_RNDN);
-      return true;
+      mpz_neg(val, uval);
+      break;
+
     case OPERATOR_NOT:
+      mpz_set_ui(val, mpz_cmp_si(uval, 0) == 0 ? 1 : 0);
+      break;
+
     case OPERATOR_XOR:
-    case OPERATOR_AND:
-    case OPERATOR_MULT:
-      return false;
+      {
+       Type* utype = unc->type();
+       if (utype->integer_type() == NULL
+           || utype->integer_type()->is_abstract())
+         mpz_com(val, uval);
+       else
+         {
+           // The number of HOST_WIDE_INTs that it takes to represent
+           // UVAL.
+           size_t count = ((mpz_sizeinbase(uval, 2)
+                            + HOST_BITS_PER_WIDE_INT
+                            - 1)
+                           / HOST_BITS_PER_WIDE_INT);
+
+           unsigned HOST_WIDE_INT* phwi = new unsigned HOST_WIDE_INT[count];
+           memset(phwi, 0, count * sizeof(HOST_WIDE_INT));
+
+           size_t obits = utype->integer_type()->bits();
+
+           if (!utype->integer_type()->is_unsigned() && mpz_sgn(uval) < 0)
+             {
+               mpz_t adj;
+               mpz_init_set_ui(adj, 1);
+               mpz_mul_2exp(adj, adj, obits);
+               mpz_add(uval, uval, adj);
+               mpz_clear(adj);
+             }
+
+           size_t ecount;
+           mpz_export(phwi, &ecount, -1, sizeof(HOST_WIDE_INT), 0, 0, uval);
+           go_assert(ecount <= count);
+
+           // Trim down to the number of words required by the type.
+           size_t ocount = ((obits + HOST_BITS_PER_WIDE_INT - 1)
+                            / HOST_BITS_PER_WIDE_INT);
+           go_assert(ocount <= count);
+
+           for (size_t i = 0; i < ocount; ++i)
+             phwi[i] = ~phwi[i];
+
+           size_t clearbits = ocount * HOST_BITS_PER_WIDE_INT - obits;
+           if (clearbits != 0)
+             phwi[ocount - 1] &= (((unsigned HOST_WIDE_INT) (HOST_WIDE_INT) -1)
+                                  >> clearbits);
+
+           mpz_import(val, ocount, -1, sizeof(HOST_WIDE_INT), 0, 0, phwi);
+
+           if (!utype->integer_type()->is_unsigned()
+               && mpz_tstbit(val, obits - 1))
+             {
+               mpz_t adj;
+               mpz_init_set_ui(adj, 1);
+               mpz_mul_2exp(adj, adj, obits);
+               mpz_sub(val, val, adj);
+               mpz_clear(adj);
+             }
+
+           delete[] phwi;
+         }
+      }
+      break;
+
     default:
       go_unreachable();
     }
-}
 
-// Apply unary opcode OP to RVAL/IVAL, setting REAL/IMAG.  Return true
-// if this could be done, false if not.
-
-bool
-Unary_expression::eval_complex(Operator op, mpfr_t rval, mpfr_t ival,
-                              mpfr_t real, mpfr_t imag)
-{
-  switch (op)
-    {
-    case OPERATOR_PLUS:
-      mpfr_set(real, rval, GMP_RNDN);
-      mpfr_set(imag, ival, GMP_RNDN);
-      return true;
-    case OPERATOR_MINUS:
-      mpfr_neg(real, rval, GMP_RNDN);
-      mpfr_neg(imag, ival, GMP_RNDN);
-      return true;
-    case OPERATOR_NOT:
-    case OPERATOR_XOR:
-    case OPERATOR_AND:
-    case OPERATOR_MULT:
-      return false;
-    default:
-      go_unreachable();
-    }
-}
-
-// Return the integral constant value of a unary expression, if it has one.
-
-bool
-Unary_expression::do_integer_constant_value(bool iota_is_constant, mpz_t val,
-                                           Type** ptype) const
-{
-  mpz_t uval;
-  mpz_init(uval);
-  bool ret;
-  if (!this->expr_->integer_constant_value(iota_is_constant, uval, ptype))
-    ret = false;
+  if (unc->is_rune())
+    nc->set_rune(NULL, val);
   else
-    ret = Unary_expression::eval_integer(this->op_, *ptype, uval, val,
-                                        this->location());
-  mpz_clear(uval);
-  return ret;
-}
+    nc->set_int(NULL, val);
 
-// Return the floating point constant value of a unary expression, if
-// it has one.
+  mpz_clear(uval);
+  mpz_clear(val);
 
-bool
-Unary_expression::do_float_constant_value(mpfr_t val, Type** ptype) const
-{
-  mpfr_t uval;
-  mpfr_init(uval);
-  bool ret;
-  if (!this->expr_->float_constant_value(uval, ptype))
-    ret = false;
-  else
-    ret = Unary_expression::eval_float(this->op_, uval, val);
-  mpfr_clear(uval);
-  return ret;
+  return nc->set_type(unc->type(), true, location);
 }
 
-// Return the complex constant value of a unary expression, if it has
-// one.
+// Return the integral constant value of a unary expression, if it has one.
 
 bool
-Unary_expression::do_complex_constant_value(mpfr_t real, mpfr_t imag,
-                                           Type** ptype) const
+Unary_expression::do_numeric_constant_value(Numeric_constant* nc) const
 {
-  mpfr_t rval;
-  mpfr_t ival;
-  mpfr_init(rval);
-  mpfr_init(ival);
-  bool ret;
-  if (!this->expr_->complex_constant_value(rval, ival, ptype))
-    ret = false;
-  else
-    ret = Unary_expression::eval_complex(this->op_, rval, ival, real, imag);
-  mpfr_clear(rval);
-  mpfr_clear(ival);
-  return ret;
+  Numeric_constant unc;
+  if (!this->expr_->numeric_constant_value(&unc))
+    return false;
+  return Unary_expression::eval_constant(this->op_, &unc, this->location(),
+                                        nc);
 }
 
 // Return the type of a unary expression.
@@ -4434,6 +3901,10 @@ Unary_expression::do_check_types(Gogo*)
       break;
 
     case OPERATOR_NOT:
+      if (!type->is_boolean_type())
+       this->report_error(_("expected boolean type"));
+      break;
+
     case OPERATOR_XOR:
       if (type->integer_type() == NULL
          && !type->is_boolean_type())
@@ -4466,11 +3937,38 @@ Unary_expression::do_check_types(Gogo*)
 tree
 Unary_expression::do_get_tree(Translate_context* context)
 {
+  Location loc = this->location();
+
+  // Taking the address of a set-and-use-temporary expression requires
+  // setting the temporary and then taking the address.
+  if (this->op_ == OPERATOR_AND)
+    {
+      Set_and_use_temporary_expression* sut =
+       this->expr_->set_and_use_temporary_expression();
+      if (sut != NULL)
+       {
+         Temporary_statement* temp = sut->temporary();
+         Bvariable* bvar = temp->get_backend_variable(context);
+         tree var_tree = var_to_tree(bvar);
+         Expression* val = sut->expression();
+         tree val_tree = val->get_tree(context);
+         if (var_tree == error_mark_node || val_tree == error_mark_node)
+           return error_mark_node;
+         tree addr_tree = build_fold_addr_expr_loc(loc.gcc_location(),
+                                                   var_tree);
+         return build2_loc(loc.gcc_location(), COMPOUND_EXPR,
+                           TREE_TYPE(addr_tree),
+                           build2_loc(sut->location().gcc_location(),
+                                      MODIFY_EXPR, void_type_node,
+                                      var_tree, val_tree),
+                           addr_tree);
+       }
+    }
+
   tree expr = this->expr_->get_tree(context);
   if (expr == error_mark_node)
     return error_mark_node;
 
-  source_location loc = this->location();
   switch (this->op_)
     {
     case OPERATOR_PLUS:
@@ -4482,7 +3980,7 @@ Unary_expression::do_get_tree(Translate_context* context)
        tree compute_type = excess_precision_type(type);
        if (compute_type != NULL_TREE)
          expr = ::convert(compute_type, expr);
-       tree ret = fold_build1_loc(loc, NEGATE_EXPR,
+       tree ret = fold_build1_loc(loc.gcc_location(), NEGATE_EXPR,
                                   (compute_type != NULL_TREE
                                    ? compute_type
                                    : type),
@@ -4494,13 +3992,15 @@ Unary_expression::do_get_tree(Translate_context* context)
 
     case OPERATOR_NOT:
       if (TREE_CODE(TREE_TYPE(expr)) == BOOLEAN_TYPE)
-       return fold_build1_loc(loc, TRUTH_NOT_EXPR, TREE_TYPE(expr), expr);
+       return fold_build1_loc(loc.gcc_location(), TRUTH_NOT_EXPR,
+                               TREE_TYPE(expr), expr);
       else
-       return fold_build2_loc(loc, NE_EXPR, boolean_type_node, expr,
-                              build_int_cst(TREE_TYPE(expr), 0));
+       return fold_build2_loc(loc.gcc_location(), NE_EXPR, boolean_type_node,
+                               expr, build_int_cst(TREE_TYPE(expr), 0));
 
     case OPERATOR_XOR:
-      return fold_build1_loc(loc, BIT_NOT_EXPR, TREE_TYPE(expr), expr);
+      return fold_build1_loc(loc.gcc_location(), BIT_NOT_EXPR, TREE_TYPE(expr),
+                             expr);
 
     case OPERATOR_AND:
       if (!this->create_temp_)
@@ -4516,7 +4016,7 @@ Unary_expression::do_get_tree(Translate_context* context)
       // Build a decl for a constant constructor.
       if (TREE_CODE(expr) == CONSTRUCTOR && TREE_CONSTANT(expr))
        {
-         tree decl = build_decl(this->location(), VAR_DECL,
+         tree decl = build_decl(this->location().gcc_location(), VAR_DECL,
                                 create_tmp_var_name("C"), TREE_TYPE(expr));
          DECL_EXTERNAL(decl) = 0;
          TREE_PUBLIC(decl) = 0;
@@ -4532,21 +4032,53 @@ Unary_expression::do_get_tree(Translate_context* context)
 
       if (this->create_temp_
          && !TREE_ADDRESSABLE(TREE_TYPE(expr))
-         && !DECL_P(expr)
+         && (TREE_CODE(expr) == CONST_DECL || !DECL_P(expr))
          && TREE_CODE(expr) != INDIRECT_REF
          && TREE_CODE(expr) != COMPONENT_REF)
        {
-         tree tmp = create_tmp_var(TREE_TYPE(expr), get_name(expr));
-         DECL_IGNORED_P(tmp) = 1;
-         DECL_INITIAL(tmp) = expr;
-         TREE_ADDRESSABLE(tmp) = 1;
-         return build2_loc(loc, COMPOUND_EXPR,
-                           build_pointer_type(TREE_TYPE(expr)),
-                           build1_loc(loc, DECL_EXPR, void_type_node, tmp),
-                           build_fold_addr_expr_loc(loc, tmp));
+         if (current_function_decl != NULL)
+           {
+             tree tmp = create_tmp_var(TREE_TYPE(expr), get_name(expr));
+             DECL_IGNORED_P(tmp) = 1;
+             DECL_INITIAL(tmp) = expr;
+             TREE_ADDRESSABLE(tmp) = 1;
+             return build2_loc(loc.gcc_location(), COMPOUND_EXPR,
+                               build_pointer_type(TREE_TYPE(expr)),
+                               build1_loc(loc.gcc_location(), DECL_EXPR,
+                                          void_type_node, tmp),
+                               build_fold_addr_expr_loc(loc.gcc_location(),
+                                                        tmp));
+           }
+         else
+           {
+             tree tmp = build_decl(loc.gcc_location(), VAR_DECL,
+                                   create_tmp_var_name("A"), TREE_TYPE(expr));
+             DECL_EXTERNAL(tmp) = 0;
+             TREE_PUBLIC(tmp) = 0;
+             TREE_STATIC(tmp) = 1;
+             DECL_ARTIFICIAL(tmp) = 1;
+             TREE_ADDRESSABLE(tmp) = 1;
+             tree make_tmp;
+             if (!TREE_CONSTANT(expr))
+               make_tmp = fold_build2_loc(loc.gcc_location(), INIT_EXPR,
+                                          void_type_node, tmp, expr);
+             else
+               {
+                 TREE_READONLY(tmp) = 1;
+                 TREE_CONSTANT(tmp) = 1;
+                 DECL_INITIAL(tmp) = expr;
+                 make_tmp = NULL_TREE;
+               }
+             rest_of_decl_compilation(tmp, 1, 0);
+             tree addr = build_fold_addr_expr_loc(loc.gcc_location(), tmp);
+             if (make_tmp == NULL_TREE)
+               return addr;
+             return build2_loc(loc.gcc_location(), COMPOUND_EXPR,
+                               TREE_TYPE(addr), make_tmp, addr);
+           }
        }
 
-      return build_fold_addr_expr_loc(loc, expr);
+      return build_fold_addr_expr_loc(loc.gcc_location(), expr);
 
     case OPERATOR_MULT:
       {
@@ -4556,33 +4088,41 @@ Unary_expression::do_get_tree(Translate_context* context)
        // need to check for nil.  We don't bother to check for small
        // structs because we expect the system to crash on a nil
        // pointer dereference.
-       HOST_WIDE_INT s = int_size_in_bytes(TREE_TYPE(TREE_TYPE(expr)));
-       if (s == -1 || s >= 4096)
+       tree target_type_tree = TREE_TYPE(TREE_TYPE(expr));
+       if (!VOID_TYPE_P(target_type_tree))
          {
-           if (!DECL_P(expr))
-             expr = save_expr(expr);
-           tree compare = fold_build2_loc(loc, EQ_EXPR, boolean_type_node,
-                                          expr,
-                                          fold_convert(TREE_TYPE(expr),
-                                                       null_pointer_node));
-           tree crash = Gogo::runtime_error(RUNTIME_ERROR_NIL_DEREFERENCE,
-                                            loc);
-           expr = fold_build2_loc(loc, COMPOUND_EXPR, TREE_TYPE(expr),
-                                  build3(COND_EXPR, void_type_node,
-                                         compare, crash, NULL_TREE),
-                                  expr);
+           HOST_WIDE_INT s = int_size_in_bytes(target_type_tree);
+           if (s == -1 || s >= 4096)
+             {
+               if (!DECL_P(expr))
+                 expr = save_expr(expr);
+               tree compare = fold_build2_loc(loc.gcc_location(), EQ_EXPR,
+                                              boolean_type_node,
+                                              expr,
+                                              fold_convert(TREE_TYPE(expr),
+                                                           null_pointer_node));
+               tree crash = Gogo::runtime_error(RUNTIME_ERROR_NIL_DEREFERENCE,
+                                                loc);
+               expr = fold_build2_loc(loc.gcc_location(), COMPOUND_EXPR,
+                                      TREE_TYPE(expr), build3(COND_EXPR,
+                                                              void_type_node,
+                                                              compare, crash,
+                                                              NULL_TREE),
+                                      expr);
+             }
          }
 
        // If the type of EXPR is a recursive pointer type, then we
        // need to insert a cast before indirecting.
-       if (TREE_TYPE(TREE_TYPE(expr)) == ptr_type_node)
+       if (VOID_TYPE_P(target_type_tree))
          {
            Type* pt = this->expr_->type()->points_to();
            tree ind = type_to_tree(pt->get_backend(context->gogo()));
-           expr = fold_convert_loc(loc, build_pointer_type(ind), expr);
+           expr = fold_convert_loc(loc.gcc_location(),
+                                    build_pointer_type(ind), expr);
          }
 
-       return build_fold_indirect_ref_loc(loc, expr);
+       return build_fold_indirect_ref_loc(loc.gcc_location(), expr);
       }
 
     default:
@@ -4659,7 +4199,7 @@ Unary_expression::do_dump_expression(Ast_dump_context* ast_dump_context) const
 // Make a unary expression.
 
 Expression*
-Expression::make_unary(Operator op, Expression* expr, source_location location)
+Expression::make_unary(Operator op, Expression* expr, Location location)
 {
   return new Unary_expression(op, expr, location);
 }
@@ -4692,124 +4232,239 @@ Binary_expression::do_traverse(Traverse* traverse)
   return Expression::traverse(&this->right_, traverse);
 }
 
-// Compare integer constants according to OP.
+// Return the type to use for a binary operation on operands of
+// LEFT_TYPE and RIGHT_TYPE.  These are the types of constants and as
+// such may be NULL or abstract.
+
+bool
+Binary_expression::operation_type(Operator op, Type* left_type,
+                                 Type* right_type, Type** result_type)
+{
+  if (left_type != right_type
+      && !left_type->is_abstract()
+      && !right_type->is_abstract()
+      && left_type->base() != right_type->base()
+      && op != OPERATOR_LSHIFT
+      && op != OPERATOR_RSHIFT)
+    {
+      // May be a type error--let it be diagnosed elsewhere.
+      return false;
+    }
+
+  if (op == OPERATOR_LSHIFT || op == OPERATOR_RSHIFT)
+    {
+      if (left_type->integer_type() != NULL)
+       *result_type = left_type;
+      else
+       *result_type = Type::make_abstract_integer_type();
+    }
+  else if (!left_type->is_abstract() && left_type->named_type() != NULL)
+    *result_type = left_type;
+  else if (!right_type->is_abstract() && right_type->named_type() != NULL)
+    *result_type = right_type;
+  else if (!left_type->is_abstract())
+    *result_type = left_type;
+  else if (!right_type->is_abstract())
+    *result_type = right_type;
+  else if (left_type->complex_type() != NULL)
+    *result_type = left_type;
+  else if (right_type->complex_type() != NULL)
+    *result_type = right_type;
+  else if (left_type->float_type() != NULL)
+    *result_type = left_type;
+  else if (right_type->float_type() != NULL)
+    *result_type = right_type;
+  else if (left_type->integer_type() != NULL
+          && left_type->integer_type()->is_rune())
+    *result_type = left_type;
+  else if (right_type->integer_type() != NULL
+          && right_type->integer_type()->is_rune())
+    *result_type = right_type;
+  else
+    *result_type = left_type;
+
+  return true;
+}
+
+// Convert an integer comparison code and an operator to a boolean
+// value.
 
 bool
-Binary_expression::compare_integer(Operator op, mpz_t left_val,
-                                  mpz_t right_val)
+Binary_expression::cmp_to_bool(Operator op, int cmp)
 {
-  int i = mpz_cmp(left_val, right_val);
   switch (op)
     {
     case OPERATOR_EQEQ:
-      return i == 0;
+      return cmp == 0;
+      break;
     case OPERATOR_NOTEQ:
-      return i != 0;
+      return cmp != 0;
+      break;
     case OPERATOR_LT:
-      return i < 0;
+      return cmp < 0;
+      break;
     case OPERATOR_LE:
-      return i <= 0;
+      return cmp <= 0;
     case OPERATOR_GT:
-      return i > 0;
+      return cmp > 0;
     case OPERATOR_GE:
-      return i >= 0;
+      return cmp >= 0;
     default:
       go_unreachable();
     }
 }
 
-// Compare floating point constants according to OP.
+// Compare constants according to OP.
 
 bool
-Binary_expression::compare_float(Operator op, Type* type, mpfr_t left_val,
-                                mpfr_t right_val)
+Binary_expression::compare_constant(Operator op, Numeric_constant* left_nc,
+                                   Numeric_constant* right_nc,
+                                   Location location, bool* result)
 {
-  int i;
-  if (type == NULL)
-    i = mpfr_cmp(left_val, right_val);
+  Type* left_type = left_nc->type();
+  Type* right_type = right_nc->type();
+
+  Type* type;
+  if (!Binary_expression::operation_type(op, left_type, right_type, &type))
+    return false;
+
+  // When comparing an untyped operand to a typed operand, we are
+  // effectively coercing the untyped operand to the other operand's
+  // type, so make sure that is valid.
+  if (!left_nc->set_type(type, true, location)
+      || !right_nc->set_type(type, true, location))
+    return false;
+
+  bool ret;
+  int cmp;
+  if (type->complex_type() != NULL)
+    {
+      if (op != OPERATOR_EQEQ && op != OPERATOR_NOTEQ)
+       return false;
+      ret = Binary_expression::compare_complex(left_nc, right_nc, &cmp);
+    }
+  else if (type->float_type() != NULL)
+    ret = Binary_expression::compare_float(left_nc, right_nc, &cmp);
   else
+    ret = Binary_expression::compare_integer(left_nc, right_nc, &cmp);
+
+  if (ret)
+    *result = Binary_expression::cmp_to_bool(op, cmp);
+
+  return ret;
+}
+
+// Compare integer constants.
+
+bool
+Binary_expression::compare_integer(const Numeric_constant* left_nc,
+                                  const Numeric_constant* right_nc,
+                                  int* cmp)
+{
+  mpz_t left_val;
+  if (!left_nc->to_int(&left_val))
+    return false;
+  mpz_t right_val;
+  if (!right_nc->to_int(&right_val))
     {
-      mpfr_t lv;
-      mpfr_init_set(lv, left_val, GMP_RNDN);
-      mpfr_t rv;
-      mpfr_init_set(rv, right_val, GMP_RNDN);
-      Float_expression::constrain_float(lv, type);
-      Float_expression::constrain_float(rv, type);
-      i = mpfr_cmp(lv, rv);
-      mpfr_clear(lv);
-      mpfr_clear(rv);
+      mpz_clear(left_val);
+      return false;
     }
-  switch (op)
+
+  *cmp = mpz_cmp(left_val, right_val);
+
+  mpz_clear(left_val);
+  mpz_clear(right_val);
+
+  return true;
+}
+
+// Compare floating point constants.
+
+bool
+Binary_expression::compare_float(const Numeric_constant* left_nc,
+                                const Numeric_constant* right_nc,
+                                int* cmp)
+{
+  mpfr_t left_val;
+  if (!left_nc->to_float(&left_val))
+    return false;
+  mpfr_t right_val;
+  if (!right_nc->to_float(&right_val))
     {
-    case OPERATOR_EQEQ:
-      return i == 0;
-    case OPERATOR_NOTEQ:
-      return i != 0;
-    case OPERATOR_LT:
-      return i < 0;
-    case OPERATOR_LE:
-      return i <= 0;
-    case OPERATOR_GT:
-      return i > 0;
-    case OPERATOR_GE:
-      return i >= 0;
-    default:
-      go_unreachable();
+      mpfr_clear(left_val);
+      return false;
+    }
+
+  // We already coerced both operands to the same type.  If that type
+  // is not an abstract type, we need to round the values accordingly.
+  Type* type = left_nc->type();
+  if (!type->is_abstract() && type->float_type() != NULL)
+    {
+      int bits = type->float_type()->bits();
+      mpfr_prec_round(left_val, bits, GMP_RNDN);
+      mpfr_prec_round(right_val, bits, GMP_RNDN);
     }
+
+  *cmp = mpfr_cmp(left_val, right_val);
+
+  mpfr_clear(left_val);
+  mpfr_clear(right_val);
+
+  return true;
 }
 
-// Compare complex constants according to OP.  Complex numbers may
-// only be compared for equality.
+// Compare complex constants.  Complex numbers may only be compared
+// for equality.
 
 bool
-Binary_expression::compare_complex(Operator op, Type* type,
-                                  mpfr_t left_real, mpfr_t left_imag,
-                                  mpfr_t right_real, mpfr_t right_imag)
+Binary_expression::compare_complex(const Numeric_constant* left_nc,
+                                  const Numeric_constant* right_nc,
+                                  int* cmp)
 {
-  bool is_equal;
-  if (type == NULL)
-    is_equal = (mpfr_cmp(left_real, right_real) == 0
-               && mpfr_cmp(left_imag, right_imag) == 0);
-  else
+  mpfr_t left_real, left_imag;
+  if (!left_nc->to_complex(&left_real, &left_imag))
+    return false;
+  mpfr_t right_real, right_imag;
+  if (!right_nc->to_complex(&right_real, &right_imag))
     {
-      mpfr_t lr;
-      mpfr_t li;
-      mpfr_init_set(lr, left_real, GMP_RNDN);
-      mpfr_init_set(li, left_imag, GMP_RNDN);
-      mpfr_t rr;
-      mpfr_t ri;
-      mpfr_init_set(rr, right_real, GMP_RNDN);
-      mpfr_init_set(ri, right_imag, GMP_RNDN);
-      Complex_expression::constrain_complex(lr, li, type);
-      Complex_expression::constrain_complex(rr, ri, type);
-      is_equal = mpfr_cmp(lr, rr) == 0 && mpfr_cmp(li, ri) == 0;
-      mpfr_clear(lr);
-      mpfr_clear(li);
-      mpfr_clear(rr);
-      mpfr_clear(ri);
+      mpfr_clear(left_real);
+      mpfr_clear(left_imag);
+      return false;
     }
-  switch (op)
+
+  // We already coerced both operands to the same type.  If that type
+  // is not an abstract type, we need to round the values accordingly.
+  Type* type = left_nc->type();
+  if (!type->is_abstract() && type->complex_type() != NULL)
     {
-    case OPERATOR_EQEQ:
-      return is_equal;
-    case OPERATOR_NOTEQ:
-      return !is_equal;
-    default:
-      go_unreachable();
+      int bits = type->complex_type()->bits();
+      mpfr_prec_round(left_real, bits / 2, GMP_RNDN);
+      mpfr_prec_round(left_imag, bits / 2, GMP_RNDN);
+      mpfr_prec_round(right_real, bits / 2, GMP_RNDN);
+      mpfr_prec_round(right_imag, bits / 2, GMP_RNDN);
     }
+
+  *cmp = (mpfr_cmp(left_real, right_real) != 0
+         || mpfr_cmp(left_imag, right_imag) != 0);
+
+  mpfr_clear(left_real);
+  mpfr_clear(left_imag);
+  mpfr_clear(right_real);
+  mpfr_clear(right_imag);
+
+  return true;
 }
 
-// Apply binary opcode OP to LEFT_VAL and RIGHT_VAL, setting VAL.
-// LEFT_TYPE is the type of LEFT_VAL, RIGHT_TYPE is the type of
-// RIGHT_VAL; LEFT_TYPE and/or RIGHT_TYPE may be NULL.  Return true if
-// this could be done, false if not.
+// Apply binary opcode OP to LEFT_NC and RIGHT_NC, setting NC.  Return
+// true if this could be done, false if not.  Issue errors at LOCATION
+// as appropriate.
 
 bool
-Binary_expression::eval_integer(Operator op, Type* left_type, mpz_t left_val,
-                               Type* right_type, mpz_t right_val,
-                               source_location location, mpz_t val)
+Binary_expression::eval_constant(Operator op, Numeric_constant* left_nc,
+                                Numeric_constant* right_nc,
+                                Location location, Numeric_constant* nc)
 {
-  bool is_shift_op = false;
   switch (op)
     {
     case OPERATOR_OROR:
@@ -4820,9 +4475,67 @@ Binary_expression::eval_integer(Operator op, Type* left_type, mpz_t left_val,
     case OPERATOR_LE:
     case OPERATOR_GT:
     case OPERATOR_GE:
-      // These return boolean values.  We should probably handle them
-      // anyhow in case a type conversion is used on the result.
+      // These return boolean values, not numeric.
+      return false;
+    default:
+      break;
+    }
+
+  Type* left_type = left_nc->type();
+  Type* right_type = right_nc->type();
+
+  Type* type;
+  if (!Binary_expression::operation_type(op, left_type, right_type, &type))
+    return false;
+
+  bool is_shift = op == OPERATOR_LSHIFT || op == OPERATOR_RSHIFT;
+
+  // When combining an untyped operand with a typed operand, we are
+  // effectively coercing the untyped operand to the other operand's
+  // type, so make sure that is valid.
+  if (!left_nc->set_type(type, true, location))
+    return false;
+  if (!is_shift && !right_nc->set_type(type, true, location))
+    return false;
+
+  bool r;
+  if (type->complex_type() != NULL)
+    r = Binary_expression::eval_complex(op, left_nc, right_nc, location, nc);
+  else if (type->float_type() != NULL)
+    r = Binary_expression::eval_float(op, left_nc, right_nc, location, nc);
+  else
+    r = Binary_expression::eval_integer(op, left_nc, right_nc, location, nc);
+
+  if (r)
+    r = nc->set_type(type, true, location);
+
+  return r;
+}
+
+// Apply binary opcode OP to LEFT_NC and RIGHT_NC, setting NC, using
+// integer operations.  Return true if this could be done, false if
+// not.
+
+bool
+Binary_expression::eval_integer(Operator op, const Numeric_constant* left_nc,
+                               const Numeric_constant* right_nc,
+                               Location location, Numeric_constant* nc)
+{
+  mpz_t left_val;
+  if (!left_nc->to_int(&left_val))
+    return false;
+  mpz_t right_val;
+  if (!right_nc->to_int(&right_val))
+    {
+      mpz_clear(left_val);
       return false;
+    }
+
+  mpz_t val;
+  mpz_init(val);
+
+  switch (op)
+    {
     case OPERATOR_PLUS:
       mpz_add(val, left_val, right_val);
       break;
@@ -4845,7 +4558,6 @@ Binary_expression::eval_integer(Operator op, Type* left_type, mpz_t left_val,
        {
          error_at(location, "division by zero");
          mpz_set_ui(val, 0);
-         return true;
        }
       break;
     case OPERATOR_MOD:
@@ -4855,20 +4567,18 @@ Binary_expression::eval_integer(Operator op, Type* left_type, mpz_t left_val,
        {
          error_at(location, "division by zero");
          mpz_set_ui(val, 0);
-         return true;
        }
       break;
     case OPERATOR_LSHIFT:
       {
        unsigned long shift = mpz_get_ui(right_val);
-       if (mpz_cmp_ui(right_val, shift) != 0 || shift > 0x100000)
+       if (mpz_cmp_ui(right_val, shift) == 0 && shift <= 0x100000)
+         mpz_mul_2exp(val, left_val, shift);
+       else
          {
            error_at(location, "shift count overflow");
            mpz_set_ui(val, 0);
-           return true;
          }
-       mpz_mul_2exp(val, left_val, shift);
-       is_shift_op = true;
        break;
       }
       break;
@@ -4879,13 +4589,14 @@ Binary_expression::eval_integer(Operator op, Type* left_type, mpz_t left_val,
          {
            error_at(location, "shift count overflow");
            mpz_set_ui(val, 0);
-           return true;
          }
-       if (mpz_cmp_ui(left_val, 0) >= 0)
-         mpz_tdiv_q_2exp(val, left_val, shift);
        else
-         mpz_fdiv_q_2exp(val, left_val, shift);
-       is_shift_op = true;
+         {
+           if (mpz_cmp_ui(left_val, 0) >= 0)
+             mpz_tdiv_q_2exp(val, left_val, shift);
+           else
+             mpz_fdiv_q_2exp(val, left_val, shift);
+         }
        break;
       }
       break;
@@ -4905,63 +4616,47 @@ Binary_expression::eval_integer(Operator op, Type* left_type, mpz_t left_val,
       go_unreachable();
     }
 
-  Type* type = left_type;
-  if (!is_shift_op)
-    {
-      if (type == NULL)
-       type = right_type;
-      else if (type != right_type && right_type != NULL)
-       {
-         if (type->is_abstract())
-           type = right_type;
-         else if (!right_type->is_abstract())
-           {
-             // This look like a type error which should be diagnosed
-             // elsewhere.  Don't do anything here, to avoid an
-             // unhelpful chain of error messages.
-             return true;
-           }
-       }
-    }
+  mpz_clear(left_val);
+  mpz_clear(right_val);
 
-  if (type != NULL && !type->is_abstract())
-    {
-      // We have to check the operands too, as we have implicitly
-      // coerced them to TYPE.
-      if ((type != left_type
-          && !Integer_expression::check_constant(left_val, type, location))
-         || (!is_shift_op
-             && type != right_type
-             && !Integer_expression::check_constant(right_val, type,
-                                                    location))
-         || !Integer_expression::check_constant(val, type, location))
-       mpz_set_ui(val, 0);
-    }
+  if (left_nc->is_rune()
+      || (op != OPERATOR_LSHIFT
+         && op != OPERATOR_RSHIFT
+         && right_nc->is_rune()))
+    nc->set_rune(NULL, val);
+  else
+    nc->set_int(NULL, val);
+
+  mpz_clear(val);
 
   return true;
 }
 
-// Apply binary opcode OP to LEFT_VAL and RIGHT_VAL, setting VAL.
-// Return true if this could be done, false if not.
+// Apply binary opcode OP to LEFT_NC and RIGHT_NC, setting NC, using
+// floating point operations.  Return true if this could be done,
+// false if not.
 
 bool
-Binary_expression::eval_float(Operator op, Type* left_type, mpfr_t left_val,
-                             Type* right_type, mpfr_t right_val,
-                             mpfr_t val, source_location location)
+Binary_expression::eval_float(Operator op, const Numeric_constant* left_nc,
+                             const Numeric_constant* right_nc,
+                             Location location, Numeric_constant* nc)
 {
-  switch (op)
+  mpfr_t left_val;
+  if (!left_nc->to_float(&left_val))
+    return false;
+  mpfr_t right_val;
+  if (!right_nc->to_float(&right_val))
     {
-    case OPERATOR_OROR:
-    case OPERATOR_ANDAND:
-    case OPERATOR_EQEQ:
-    case OPERATOR_NOTEQ:
-    case OPERATOR_LT:
-    case OPERATOR_LE:
-    case OPERATOR_GT:
-    case OPERATOR_GE:
-      // These return boolean values.  We should probably handle them
-      // anyhow in case a type conversion is used on the result.
+      mpfr_clear(left_val);
       return false;
+    }
+
+  mpfr_t val;
+  mpfr_init(val);
+
+  bool ret = true;
+  switch (op)
+    {
     case OPERATOR_PLUS:
       mpfr_add(val, left_val, right_val, GMP_RNDN);
       break;
@@ -4972,78 +4667,64 @@ Binary_expression::eval_float(Operator op, Type* left_type, mpfr_t left_val,
     case OPERATOR_XOR:
     case OPERATOR_AND:
     case OPERATOR_BITCLEAR:
-      return false;
+    case OPERATOR_MOD:
+    case OPERATOR_LSHIFT:
+    case OPERATOR_RSHIFT:
+      mpfr_set_ui(val, 0, GMP_RNDN);
+      ret = false;
+      break;
     case OPERATOR_MULT:
       mpfr_mul(val, left_val, right_val, GMP_RNDN);
       break;
     case OPERATOR_DIV:
-      if (mpfr_zero_p(right_val))
-       error_at(location, "division by zero");
-      mpfr_div(val, left_val, right_val, GMP_RNDN);
+      if (!mpfr_zero_p(right_val))
+       mpfr_div(val, left_val, right_val, GMP_RNDN);
+      else
+       {
+         error_at(location, "division by zero");
+         mpfr_set_ui(val, 0, GMP_RNDN);
+       }
       break;
-    case OPERATOR_MOD:
-      return false;
-    case OPERATOR_LSHIFT:
-    case OPERATOR_RSHIFT:
-      return false;
     default:
       go_unreachable();
     }
 
-  Type* type = left_type;
-  if (type == NULL)
-    type = right_type;
-  else if (type != right_type && right_type != NULL)
-    {
-      if (type->is_abstract())
-       type = right_type;
-      else if (!right_type->is_abstract())
-       {
-         // This looks like a type error which should be diagnosed
-         // elsewhere.  Don't do anything here, to avoid an unhelpful
-         // chain of error messages.
-         return true;
-       }
-    }
+  mpfr_clear(left_val);
+  mpfr_clear(right_val);
 
-  if (type != NULL && !type->is_abstract())
-    {
-      if ((type != left_type
-          && !Float_expression::check_constant(left_val, type, location))
-         || (type != right_type
-             && !Float_expression::check_constant(right_val, type,
-                                                  location))
-         || !Float_expression::check_constant(val, type, location))
-       mpfr_set_ui(val, 0, GMP_RNDN);
-    }
+  nc->set_float(NULL, val);
+  mpfr_clear(val);
 
-  return true;
+  return ret;
 }
 
-// Apply binary opcode OP to LEFT_REAL/LEFT_IMAG and
-// RIGHT_REAL/RIGHT_IMAG, setting REAL/IMAG.  Return true if this
-// could be done, false if not.
+// Apply binary opcode OP to LEFT_NC and RIGHT_NC, setting NC, using
+// complex operations.  Return true if this could be done, false if
+// not.
 
 bool
-Binary_expression::eval_complex(Operator op, Type* left_type,
-                               mpfr_t left_real, mpfr_t left_imag,
-                               Type *right_type,
-                               mpfr_t right_real, mpfr_t right_imag,
-                               mpfr_t real, mpfr_t imag,
-                               source_location location)
+Binary_expression::eval_complex(Operator op, const Numeric_constant* left_nc,
+                               const Numeric_constant* right_nc,
+                               Location location, Numeric_constant* nc)
 {
-  switch (op)
+  mpfr_t left_real, left_imag;
+  if (!left_nc->to_complex(&left_real, &left_imag))
+    return false;
+  mpfr_t right_real, right_imag;
+  if (!right_nc->to_complex(&right_real, &right_imag))
     {
-    case OPERATOR_OROR:
-    case OPERATOR_ANDAND:
-    case OPERATOR_EQEQ:
-    case OPERATOR_NOTEQ:
-    case OPERATOR_LT:
-    case OPERATOR_LE:
-    case OPERATOR_GT:
-    case OPERATOR_GE:
-      // These return boolean values and must be handled differently.
+      mpfr_clear(left_real);
+      mpfr_clear(left_imag);
       return false;
+    }
+
+  mpfr_t real, imag;
+  mpfr_init(real);
+  mpfr_init(imag);
+
+  bool ret = true;
+  switch (op)
+    {
     case OPERATOR_PLUS:
       mpfr_add(real, left_real, right_real, GMP_RNDN);
       mpfr_add(imag, left_imag, right_imag, GMP_RNDN);
@@ -5056,7 +4737,13 @@ Binary_expression::eval_complex(Operator op, Type* left_type,
     case OPERATOR_XOR:
     case OPERATOR_AND:
     case OPERATOR_BITCLEAR:
-      return false;
+    case OPERATOR_MOD:
+    case OPERATOR_LSHIFT:
+    case OPERATOR_RSHIFT:
+      mpfr_set_ui(real, 0, GMP_RNDN);
+      mpfr_set_ui(imag, 0, GMP_RNDN);
+      ret = false;
+      break;
     case OPERATOR_MULT:
       {
        // You might think that multiplying two complex numbers would
@@ -5209,7 +4896,12 @@ Binary_expression::eval_complex(Operator op, Type* left_type,
        // scale the values to try to avoid this.
 
        if (mpfr_zero_p(right_real) && mpfr_zero_p(right_imag))
-         error_at(location, "division by zero");
+         {
+           error_at(location, "division by zero");
+           mpfr_set_ui(real, 0, GMP_RNDN);
+           mpfr_set_ui(imag, 0, GMP_RNDN);
+           break;
+         }
 
        mpfr_t rra;
        mpfr_t ria;
@@ -5340,48 +5032,20 @@ Binary_expression::eval_complex(Operator op, Type* left_type,
        mpfr_clear(ria);
       }
       break;
-    case OPERATOR_MOD:
-      return false;
-    case OPERATOR_LSHIFT:
-    case OPERATOR_RSHIFT:
-      return false;
     default:
       go_unreachable();
     }
 
-  Type* type = left_type;
-  if (type == NULL)
-    type = right_type;
-  else if (type != right_type && right_type != NULL)
-    {
-      if (type->is_abstract())
-       type = right_type;
-      else if (!right_type->is_abstract())
-       {
-         // This looks like a type error which should be diagnosed
-         // elsewhere.  Don't do anything here, to avoid an unhelpful
-         // chain of error messages.
-         return true;
-       }
-    }
+  mpfr_clear(left_real);
+  mpfr_clear(left_imag);
+  mpfr_clear(right_real);
+  mpfr_clear(right_imag);
 
-  if (type != NULL && !type->is_abstract())
-    {
-      if ((type != left_type
-          && !Complex_expression::check_constant(left_real, left_imag,
-                                                 type, location))
-         || (type != right_type
-             && !Complex_expression::check_constant(right_real, right_imag,
-                                                    type, location))
-         || !Complex_expression::check_constant(real, imag, type,
-                                                location))
-       {
-         mpfr_set_ui(real, 0, GMP_RNDN);
-         mpfr_set_ui(imag, 0, GMP_RNDN);
-       }
-    }
+  nc->set_complex(NULL, real, imag);
+  mpfr_clear(real);
+  mpfr_clear(imag);
 
-  return true;
+  return ret;
 }
 
 // Lower a binary expression.  We have to evaluate constant
@@ -5389,9 +5053,10 @@ Binary_expression::eval_complex(Operator op, Type* left_type,
 // constants.
 
 Expression*
-Binary_expression::do_lower(Gogo*, Named_object*, Statement_inserter*, int)
+Binary_expression::do_lower(Gogo* gogo, Named_object*,
+                           Statement_inserter* inserter, int)
 {
-  source_location location = this->location();
+  Location location = this->location();
   Operator op = this->op_;
   Expression* left = this->left_;
   Expression* right = this->right_;
@@ -5403,472 +5068,249 @@ Binary_expression::do_lower(Gogo*, Named_object*, Statement_inserter*, int)
                              || op == OPERATOR_GT
                              || op == OPERATOR_GE);
 
-  // Integer constant expressions.
-  {
-    mpz_t left_val;
-    mpz_init(left_val);
-    Type* left_type;
-    mpz_t right_val;
-    mpz_init(right_val);
-    Type* right_type;
-    if (left->integer_constant_value(false, left_val, &left_type)
-       && right->integer_constant_value(false, right_val, &right_type))
-      {
-       Expression* ret = NULL;
-       if (left_type != right_type
-           && left_type != NULL
-           && right_type != NULL
-           && left_type->base() != right_type->base()
-           && op != OPERATOR_LSHIFT
-           && op != OPERATOR_RSHIFT)
-         {
-           // May be a type error--let it be diagnosed later.
-         }
-       else if (is_comparison)
-         {
-           bool b = Binary_expression::compare_integer(op, left_val,
-                                                       right_val);
-           ret = Expression::make_cast(Type::lookup_bool_type(),
-                                       Expression::make_boolean(b, location),
-                                       location);
-         }
-       else
-         {
-           mpz_t val;
-           mpz_init(val);
-
-           if (Binary_expression::eval_integer(op, left_type, left_val,
-                                               right_type, right_val,
-                                               location, val))
-             {
-               go_assert(op != OPERATOR_OROR && op != OPERATOR_ANDAND);
-               Type* type;
-               if (op == OPERATOR_LSHIFT || op == OPERATOR_RSHIFT)
-                 type = left_type;
-               else if (left_type == NULL)
-                 type = right_type;
-               else if (right_type == NULL)
-                 type = left_type;
-               else if (!left_type->is_abstract()
-                        && left_type->named_type() != NULL)
-                 type = left_type;
-               else if (!right_type->is_abstract()
-                        && right_type->named_type() != NULL)
-                 type = right_type;
-               else if (!left_type->is_abstract())
-                 type = left_type;
-               else if (!right_type->is_abstract())
-                 type = right_type;
-               else if (left_type->float_type() != NULL)
-                 type = left_type;
-               else if (right_type->float_type() != NULL)
-                 type = right_type;
-               else if (left_type->complex_type() != NULL)
-                 type = left_type;
-               else if (right_type->complex_type() != NULL)
-                 type = right_type;
-               else
-                 type = left_type;
-               ret = Expression::make_integer(&val, type, location);
-             }
-
-           mpz_clear(val);
-         }
-
-       if (ret != NULL)
-         {
-           mpz_clear(right_val);
-           mpz_clear(left_val);
-           return ret;
-         }
-      }
-    mpz_clear(right_val);
-    mpz_clear(left_val);
-  }
-
-  // Floating point constant expressions.
-  {
-    mpfr_t left_val;
-    mpfr_init(left_val);
-    Type* left_type;
-    mpfr_t right_val;
-    mpfr_init(right_val);
-    Type* right_type;
-    if (left->float_constant_value(left_val, &left_type)
-       && right->float_constant_value(right_val, &right_type))
-      {
-       Expression* ret = NULL;
-       if (left_type != right_type
-           && left_type != NULL
-           && right_type != NULL
-           && left_type->base() != right_type->base()
-           && op != OPERATOR_LSHIFT
-           && op != OPERATOR_RSHIFT)
-         {
-           // May be a type error--let it be diagnosed later.
-         }
-       else if (is_comparison)
-         {
-           bool b = Binary_expression::compare_float(op,
-                                                     (left_type != NULL
-                                                      ? left_type
-                                                      : right_type),
-                                                     left_val, right_val);
-           ret = Expression::make_boolean(b, location);
-         }
-       else
-         {
-           mpfr_t val;
-           mpfr_init(val);
-
-           if (Binary_expression::eval_float(op, left_type, left_val,
-                                             right_type, right_val, val,
-                                             location))
-             {
-               go_assert(op != OPERATOR_OROR && op != OPERATOR_ANDAND
-                          && op != OPERATOR_LSHIFT && op != OPERATOR_RSHIFT);
-               Type* type;
-               if (left_type == NULL)
-                 type = right_type;
-               else if (right_type == NULL)
-                 type = left_type;
-               else if (!left_type->is_abstract()
-                        && left_type->named_type() != NULL)
-                 type = left_type;
-               else if (!right_type->is_abstract()
-                        && right_type->named_type() != NULL)
-                 type = right_type;
-               else if (!left_type->is_abstract())
-                 type = left_type;
-               else if (!right_type->is_abstract())
-                 type = right_type;
-               else if (left_type->float_type() != NULL)
-                 type = left_type;
-               else if (right_type->float_type() != NULL)
-                 type = right_type;
-               else
-                 type = left_type;
-               ret = Expression::make_float(&val, type, location);
-             }
-
-           mpfr_clear(val);
-         }
-
-       if (ret != NULL)
-         {
-           mpfr_clear(right_val);
-           mpfr_clear(left_val);
-           return ret;
-         }
-      }
-    mpfr_clear(right_val);
-    mpfr_clear(left_val);
-  }
-
-  // Complex constant expressions.
+  // Numeric constant expressions.
   {
-    mpfr_t left_real;
-    mpfr_t left_imag;
-    mpfr_init(left_real);
-    mpfr_init(left_imag);
-    Type* left_type;
-
-    mpfr_t right_real;
-    mpfr_t right_imag;
-    mpfr_init(right_real);
-    mpfr_init(right_imag);
-    Type* right_type;
-
-    if (left->complex_constant_value(left_real, left_imag, &left_type)
-       && right->complex_constant_value(right_real, right_imag, &right_type))
+    Numeric_constant left_nc;
+    Numeric_constant right_nc;
+    if (left->numeric_constant_value(&left_nc)
+       && right->numeric_constant_value(&right_nc))
       {
-       Expression* ret = NULL;
-       if (left_type != right_type
-           && left_type != NULL
-           && right_type != NULL
-           && left_type->base() != right_type->base())
-         {
-           // May be a type error--let it be diagnosed later.
-         }
-       else if (op == OPERATOR_EQEQ || op == OPERATOR_NOTEQ)
+       if (is_comparison)
          {
-           bool b = Binary_expression::compare_complex(op,
-                                                       (left_type != NULL
-                                                        ? left_type
-                                                        : right_type),
-                                                       left_real,
-                                                       left_imag,
-                                                       right_real,
-                                                       right_imag);
-           ret = Expression::make_boolean(b, location);
+           bool result;
+           if (!Binary_expression::compare_constant(op, &left_nc,
+                                                    &right_nc, location,
+                                                    &result))
+             return this;
+           return Expression::make_cast(Type::lookup_bool_type(),
+                                        Expression::make_boolean(result,
+                                                                 location),
+                                        location);
          }
        else
          {
-           mpfr_t real;
-           mpfr_t imag;
-           mpfr_init(real);
-           mpfr_init(imag);
-
-           if (Binary_expression::eval_complex(op, left_type,
-                                               left_real, left_imag,
-                                               right_type,
-                                               right_real, right_imag,
-                                               real, imag,
-                                               location))
-             {
-               go_assert(op != OPERATOR_OROR && op != OPERATOR_ANDAND
-                          && op != OPERATOR_LSHIFT && op != OPERATOR_RSHIFT);
-               Type* type;
-               if (left_type == NULL)
-                 type = right_type;
-               else if (right_type == NULL)
-                 type = left_type;
-               else if (!left_type->is_abstract()
-                        && left_type->named_type() != NULL)
-                 type = left_type;
-               else if (!right_type->is_abstract()
-                        && right_type->named_type() != NULL)
-                 type = right_type;
-               else if (!left_type->is_abstract())
-                 type = left_type;
-               else if (!right_type->is_abstract())
-                 type = right_type;
-               else if (left_type->complex_type() != NULL)
-                 type = left_type;
-               else if (right_type->complex_type() != NULL)
-                 type = right_type;
-               else
-                 type = left_type;
-               ret = Expression::make_complex(&real, &imag, type,
-                                              location);
-             }
-           mpfr_clear(real);
-           mpfr_clear(imag);
-         }
-
-       if (ret != NULL)
-         {
-           mpfr_clear(left_real);
-           mpfr_clear(left_imag);
-           mpfr_clear(right_real);
-           mpfr_clear(right_imag);
-           return ret;
+           Numeric_constant nc;
+           if (!Binary_expression::eval_constant(op, &left_nc, &right_nc,
+                                                 location, &nc))
+             return this;
+           return nc.expression(location);
          }
       }
-
-    mpfr_clear(left_real);
-    mpfr_clear(left_imag);
-    mpfr_clear(right_real);
-    mpfr_clear(right_imag);
   }
 
   // String constant expressions.
-  if (op == OPERATOR_PLUS
-      && left->type()->is_string_type()
-      && right->type()->is_string_type())
+  if (left->type()->is_string_type() && right->type()->is_string_type())
     {
       std::string left_string;
       std::string right_string;
       if (left->string_constant_value(&left_string)
          && right->string_constant_value(&right_string))
-       return Expression::make_string(left_string + right_string, location);
-    }
-
-  // Special case for shift of a floating point constant.
-  if (op == OPERATOR_LSHIFT || op == OPERATOR_RSHIFT)
-    {
-      mpfr_t left_val;
-      mpfr_init(left_val);
-      Type* left_type;
-      mpz_t right_val;
-      mpz_init(right_val);
-      Type* right_type;
-      if (left->float_constant_value(left_val, &left_type)
-         && right->integer_constant_value(false, right_val, &right_type)
-         && mpfr_integer_p(left_val)
-         && (left_type == NULL
-             || left_type->is_abstract()
-             || left_type->integer_type() != NULL))
-       {
-         mpz_t left_int;
-         mpz_init(left_int);
-         mpfr_get_z(left_int, left_val, GMP_RNDN);
-
-         mpz_t val;
-         mpz_init(val);
-
-         Expression* ret = NULL;
-         if (Binary_expression::eval_integer(op, left_type, left_int,
-                                             right_type, right_val,
-                                             location, val))
-           ret = Expression::make_integer(&val, left_type, location);
-
-         mpz_clear(left_int);
-         mpz_clear(val);
-
-         if (ret != NULL)
+       {
+         if (op == OPERATOR_PLUS)
+           return Expression::make_string(left_string + right_string,
+                                          location);
+         else if (is_comparison)
            {
-             mpfr_clear(left_val);
-             mpz_clear(right_val);
-             return ret;
+             int cmp = left_string.compare(right_string);
+             bool r = Binary_expression::cmp_to_bool(op, cmp);
+             return Expression::make_cast(Type::lookup_bool_type(),
+                                          Expression::make_boolean(r,
+                                                                   location),
+                                          location);
            }
        }
+    }
 
-      mpfr_clear(left_val);
-      mpz_clear(right_val);
+  // Lower struct and array comparisons.
+  if (op == OPERATOR_EQEQ || op == OPERATOR_NOTEQ)
+    {
+      if (left->type()->struct_type() != NULL)
+       return this->lower_struct_comparison(gogo, inserter);
+      else if (left->type()->array_type() != NULL
+              && !left->type()->is_slice_type())
+       return this->lower_array_comparison(gogo, inserter);
     }
 
   return this;
 }
 
-// Return the integer constant value, if it has one.
+// Lower a struct comparison.
 
-bool
-Binary_expression::do_integer_constant_value(bool iota_is_constant, mpz_t val,
-                                            Type** ptype) const
+Expression*
+Binary_expression::lower_struct_comparison(Gogo* gogo,
+                                          Statement_inserter* inserter)
 {
-  mpz_t left_val;
-  mpz_init(left_val);
-  Type* left_type;
-  if (!this->left_->integer_constant_value(iota_is_constant, left_val,
-                                          &left_type))
+  Struct_type* st = this->left_->type()->struct_type();
+  Struct_type* st2 = this->right_->type()->struct_type();
+  if (st2 == NULL)
+    return this;
+  if (st != st2 && !Type::are_identical(st, st2, false, NULL))
+    return this;
+  if (!Type::are_compatible_for_comparison(true, this->left_->type(),
+                                          this->right_->type(), NULL))
+    return this;
+
+  // See if we can compare using memcmp.  As a heuristic, we use
+  // memcmp rather than field references and comparisons if there are
+  // more than two fields.
+  if (st->compare_is_identity(gogo) && st->total_field_count() > 2)
+    return this->lower_compare_to_memcmp(gogo, inserter);
+
+  Location loc = this->location();
+
+  Expression* left = this->left_;
+  Temporary_statement* left_temp = NULL;
+  if (left->var_expression() == NULL
+      && left->temporary_reference_expression() == NULL)
     {
-      mpz_clear(left_val);
-      return false;
+      left_temp = Statement::make_temporary(left->type(), NULL, loc);
+      inserter->insert(left_temp);
+      left = Expression::make_set_and_use_temporary(left_temp, left, loc);
     }
 
-  mpz_t right_val;
-  mpz_init(right_val);
-  Type* right_type;
-  if (!this->right_->integer_constant_value(iota_is_constant, right_val,
-                                           &right_type))
+  Expression* right = this->right_;
+  Temporary_statement* right_temp = NULL;
+  if (right->var_expression() == NULL
+      && right->temporary_reference_expression() == NULL)
     {
-      mpz_clear(right_val);
-      mpz_clear(left_val);
-      return false;
+      right_temp = Statement::make_temporary(right->type(), NULL, loc);
+      inserter->insert(right_temp);
+      right = Expression::make_set_and_use_temporary(right_temp, right, loc);
     }
 
-  bool ret;
-  if (left_type != right_type
-      && left_type != NULL
-      && right_type != NULL
-      && left_type->base() != right_type->base()
-      && this->op_ != OPERATOR_RSHIFT
-      && this->op_ != OPERATOR_LSHIFT)
-    ret = false;
-  else
-    ret = Binary_expression::eval_integer(this->op_, left_type, left_val,
-                                         right_type, right_val,
-                                         this->location(), val);
-
-  mpz_clear(right_val);
-  mpz_clear(left_val);
+  Expression* ret = Expression::make_boolean(true, loc);
+  const Struct_field_list* fields = st->fields();
+  unsigned int field_index = 0;
+  for (Struct_field_list::const_iterator pf = fields->begin();
+       pf != fields->end();
+       ++pf, ++field_index)
+    {
+      if (field_index > 0)
+       {
+         if (left_temp == NULL)
+           left = left->copy();
+         else
+           left = Expression::make_temporary_reference(left_temp, loc);
+         if (right_temp == NULL)
+           right = right->copy();
+         else
+           right = Expression::make_temporary_reference(right_temp, loc);
+       }
+      Expression* f1 = Expression::make_field_reference(left, field_index,
+                                                       loc);
+      Expression* f2 = Expression::make_field_reference(right, field_index,
+                                                       loc);
+      Expression* cond = Expression::make_binary(OPERATOR_EQEQ, f1, f2, loc);
+      ret = Expression::make_binary(OPERATOR_ANDAND, ret, cond, loc);
+    }
 
-  if (ret)
-    *ptype = left_type;
+  if (this->op_ == OPERATOR_NOTEQ)
+    ret = Expression::make_unary(OPERATOR_NOT, ret, loc);
 
   return ret;
 }
 
-// Return the floating point constant value, if it has one.
+// Lower an array comparison.
 
-bool
-Binary_expression::do_float_constant_value(mpfr_t val, Type** ptype) const
+Expression*
+Binary_expression::lower_array_comparison(Gogo* gogo,
+                                         Statement_inserter* inserter)
 {
-  mpfr_t left_val;
-  mpfr_init(left_val);
-  Type* left_type;
-  if (!this->left_->float_constant_value(left_val, &left_type))
-    {
-      mpfr_clear(left_val);
-      return false;
-    }
+  Array_type* at = this->left_->type()->array_type();
+  Array_type* at2 = this->right_->type()->array_type();
+  if (at2 == NULL)
+    return this;
+  if (at != at2 && !Type::are_identical(at, at2, false, NULL))
+    return this;
+  if (!Type::are_compatible_for_comparison(true, this->left_->type(),
+                                          this->right_->type(), NULL))
+    return this;
 
-  mpfr_t right_val;
-  mpfr_init(right_val);
-  Type* right_type;
-  if (!this->right_->float_constant_value(right_val, &right_type))
-    {
-      mpfr_clear(right_val);
-      mpfr_clear(left_val);
-      return false;
-    }
+  // Call memcmp directly if possible.  This may let the middle-end
+  // optimize the call.
+  if (at->compare_is_identity(gogo))
+    return this->lower_compare_to_memcmp(gogo, inserter);
 
-  bool ret;
-  if (left_type != right_type
-      && left_type != NULL
-      && right_type != NULL
-      && left_type->base() != right_type->base())
-    ret = false;
-  else
-    ret = Binary_expression::eval_float(this->op_, left_type, left_val,
-                                       right_type, right_val,
-                                       val, this->location());
+  // Call the array comparison function.
+  Named_object* hash_fn;
+  Named_object* equal_fn;
+  at->type_functions(gogo, this->left_->type()->named_type(), NULL, NULL,
+                    &hash_fn, &equal_fn);
 
-  mpfr_clear(left_val);
-  mpfr_clear(right_val);
+  Location loc = this->location();
 
-  if (ret)
-    *ptype = left_type;
+  Expression* func = Expression::make_func_reference(equal_fn, NULL, loc);
+
+  Expression_list* args = new Expression_list();
+  args->push_back(this->operand_address(inserter, this->left_));
+  args->push_back(this->operand_address(inserter, this->right_));
+  args->push_back(Expression::make_type_info(at, TYPE_INFO_SIZE));
+
+  Expression* ret = Expression::make_call(func, args, false, loc);
+
+  if (this->op_ == OPERATOR_NOTEQ)
+    ret = Expression::make_unary(OPERATOR_NOT, ret, loc);
 
   return ret;
 }
 
-// Return the complex constant value, if it has one.
+// Lower a struct or array comparison to a call to memcmp.
 
-bool
-Binary_expression::do_complex_constant_value(mpfr_t real, mpfr_t imag,
-                                            Type** ptype) const
+Expression*
+Binary_expression::lower_compare_to_memcmp(Gogo*, Statement_inserter* inserter)
 {
-  mpfr_t left_real;
-  mpfr_t left_imag;
-  mpfr_init(left_real);
-  mpfr_init(left_imag);
-  Type* left_type;
-  if (!this->left_->complex_constant_value(left_real, left_imag, &left_type))
-    {
-      mpfr_clear(left_real);
-      mpfr_clear(left_imag);
-      return false;
-    }
+  Location loc = this->location();
+
+  Expression* a1 = this->operand_address(inserter, this->left_);
+  Expression* a2 = this->operand_address(inserter, this->right_);
+  Expression* len = Expression::make_type_info(this->left_->type(),
+                                              TYPE_INFO_SIZE);
+
+  Expression* call = Runtime::make_call(Runtime::MEMCMP, loc, 3, a1, a2, len);
+
+  mpz_t zval;
+  mpz_init_set_ui(zval, 0);
+  Expression* zero = Expression::make_integer(&zval, NULL, loc);
+  mpz_clear(zval);
 
-  mpfr_t right_real;
-  mpfr_t right_imag;
-  mpfr_init(right_real);
-  mpfr_init(right_imag);
-  Type* right_type;
-  if (!this->right_->complex_constant_value(right_real, right_imag,
-                                           &right_type))
+  return Expression::make_binary(this->op_, call, zero, loc);
+}
+
+// Return the address of EXPR, cast to unsafe.Pointer.
+
+Expression*
+Binary_expression::operand_address(Statement_inserter* inserter,
+                                  Expression* expr)
+{
+  Location loc = this->location();
+
+  if (!expr->is_addressable())
     {
-      mpfr_clear(left_real);
-      mpfr_clear(left_imag);
-      mpfr_clear(right_real);
-      mpfr_clear(right_imag);
-      return false;
+      Temporary_statement* temp = Statement::make_temporary(expr->type(), NULL,
+                                                           loc);
+      inserter->insert(temp);
+      expr = Expression::make_set_and_use_temporary(temp, expr, loc);
     }
+  expr = Expression::make_unary(OPERATOR_AND, expr, loc);
+  static_cast<Unary_expression*>(expr)->set_does_not_escape();
+  Type* void_type = Type::make_void_type();
+  Type* unsafe_pointer_type = Type::make_pointer_type(void_type);
+  return Expression::make_cast(unsafe_pointer_type, expr, loc);
+}
 
-  bool ret;
-  if (left_type != right_type
-      && left_type != NULL
-      && right_type != NULL
-      && left_type->base() != right_type->base())
-    ret = false;
-  else
-    ret = Binary_expression::eval_complex(this->op_, left_type,
-                                         left_real, left_imag,
-                                         right_type,
-                                         right_real, right_imag,
-                                         real, imag,
-                                         this->location());
-  mpfr_clear(left_real);
-  mpfr_clear(left_imag);
-  mpfr_clear(right_real);
-  mpfr_clear(right_imag);
-
-  if (ret)
-    *ptype = left_type;
+// Return the numeric constant value, if it has one.
 
-  return ret;
+bool
+Binary_expression::do_numeric_constant_value(Numeric_constant* nc) const
+{
+  Numeric_constant left_nc;
+  if (!this->left_->numeric_constant_value(&left_nc))
+    return false;
+  Numeric_constant right_nc;
+  if (!this->right_->numeric_constant_value(&right_nc))
+    return false;
+  return Binary_expression::eval_constant(this->op_, &left_nc, &right_nc,
+                                         this->location(), nc);
 }
 
 // Note that the value is being discarded.
@@ -5912,35 +5354,13 @@ Binary_expression::do_type()
     case OPERATOR_AND:
     case OPERATOR_BITCLEAR:
       {
-       Type* left_type = this->left_->type();
-       Type* right_type = this->right_->type();
-       if (left_type->is_error())
-         return left_type;
-       else if (right_type->is_error())
-         return right_type;
-       else if (!Type::are_compatible_for_binop(left_type, right_type))
-         {
-           this->report_error(_("incompatible types in binary expression"));
-           return Type::make_error_type();
-         }
-       else if (!left_type->is_abstract() && left_type->named_type() != NULL)
-         return left_type;
-       else if (!right_type->is_abstract() && right_type->named_type() != NULL)
-         return right_type;
-       else if (!left_type->is_abstract())
-         return left_type;
-       else if (!right_type->is_abstract())
-         return right_type;
-       else if (left_type->complex_type() != NULL)
-         return left_type;
-       else if (right_type->complex_type() != NULL)
-         return right_type;
-       else if (left_type->float_type() != NULL)
-         return left_type;
-       else if (right_type->float_type() != NULL)
-         return right_type;
-       else
-         return left_type;
+       Type* type;
+       if (!Binary_expression::operation_type(this->op_,
+                                              this->left_->type(),
+                                              this->right_->type(),
+                                              &type))
+         return Type::make_error_type();
+       return type;
       }
 
     case OPERATOR_LSHIFT:
@@ -6043,12 +5463,12 @@ Binary_expression::do_determine_type(const Type_context* context)
 }
 
 // Report an error if the binary operator OP does not support TYPE.
-// Return whether the operation is OK.  This should not be used for
-// shift.
+// OTYPE is the type of the other operand.  Return whether the
+// operation is OK.  This should not be used for shift.
 
 bool
-Binary_expression::check_operator_type(Operator op, Type* type,
-                                      source_location location)
+Binary_expression::check_operator_type(Operator op, Type* type, Type* otype,
+                                      Location location)
 {
   switch (op)
     {
@@ -6063,39 +5483,28 @@ Binary_expression::check_operator_type(Operator op, Type* type,
 
     case OPERATOR_EQEQ:
     case OPERATOR_NOTEQ:
-      if (type->integer_type() == NULL
-         && type->float_type() == NULL
-         && type->complex_type() == NULL
-         && !type->is_string_type()
-         && type->points_to() == NULL
-         && !type->is_nil_type()
-         && !type->is_boolean_type()
-         && type->interface_type() == NULL
-         && (type->array_type() == NULL
-             || type->array_type()->length() != NULL)
-         && type->map_type() == NULL
-         && type->channel_type() == NULL
-         && type->function_type() == NULL)
-       {
-         error_at(location,
-                  ("expected integer, floating, complex, string, pointer, "
-                   "boolean, interface, slice, map, channel, "
-                   "or function type"));
-         return false;
-       }
+      {
+       std::string reason;
+       if (!Type::are_compatible_for_comparison(true, type, otype, &reason))
+         {
+           error_at(location, "%s", reason.c_str());
+           return false;
+         }
+      }
       break;
 
     case OPERATOR_LT:
     case OPERATOR_LE:
     case OPERATOR_GT:
     case OPERATOR_GE:
-      if (type->integer_type() == NULL
-         && type->float_type() == NULL
-         && !type->is_string_type())
-       {
-         error_at(location, "expected integer, floating, or string type");
-         return false;
-       }
+      {
+       std::string reason;
+       if (!Type::are_compatible_for_comparison(false, type, otype, &reason))
+         {
+           error_at(location, "%s", reason.c_str());
+           return false;
+         }
+      }
       break;
 
     case OPERATOR_PLUS:
@@ -6180,8 +5589,10 @@ Binary_expression::do_check_types(Gogo*)
          return;
        }
       if (!Binary_expression::check_operator_type(this->op_, left_type,
+                                                 right_type,
                                                  this->location())
          || !Binary_expression::check_operator_type(this->op_, right_type,
+                                                    left_type,
                                                     this->location()))
        {
          this->set_is_error();
@@ -6196,6 +5607,7 @@ Binary_expression::do_check_types(Gogo*)
          return;
        }
       if (!Binary_expression::check_operator_type(this->op_, left_type,
+                                                 right_type,
                                                  this->location()))
        {
          this->set_is_error();
@@ -6213,21 +5625,25 @@ Binary_expression::do_check_types(Gogo*)
        this->report_error(_("shift count not unsigned integer"));
       else
        {
-         mpz_t val;
-         mpz_init(val);
-         Type* type;
-         if (this->right_->integer_constant_value(true, val, &type))
+         Numeric_constant nc;
+         if (this->right_->numeric_constant_value(&nc))
            {
-             if (mpz_sgn(val) < 0)
+             mpz_t val;
+             if (!nc.to_int(&val))
+               this->report_error(_("shift count not unsigned integer"));
+             else
                {
-                 this->report_error(_("negative shift count"));
-                 mpz_set_ui(val, 0);
-                 source_location rloc = this->right_->location();
-                 this->right_ = Expression::make_integer(&val, right_type,
-                                                         rloc);
+                 if (mpz_sgn(val) < 0)
+                   {
+                     this->report_error(_("negative shift count"));
+                     mpz_set_ui(val, 0);
+                     Location rloc = this->right_->location();
+                     this->right_ = Expression::make_integer(&val, right_type,
+                                                             rloc);
+                   }
+                 mpz_clear(val);
                }
            }
-         mpz_clear(val);
        }
     }
 }
@@ -6246,6 +5662,7 @@ Binary_expression::do_get_tree(Translate_context* context)
   enum tree_code code;
   bool use_left_type = true;
   bool is_shift_op = false;
+  bool is_idiv_op = false;
   switch (this->op_)
     {
     case OPERATOR_EQEQ:
@@ -6288,11 +5705,15 @@ Binary_expression::do_get_tree(Translate_context* context)
        if (t->float_type() != NULL || t->complex_type() != NULL)
          code = RDIV_EXPR;
        else
-         code = TRUNC_DIV_EXPR;
+         {
+           code = TRUNC_DIV_EXPR;
+           is_idiv_op = true;
+         }
       }
       break;
     case OPERATOR_MOD:
       code = TRUNC_MOD_EXPR;
+      is_idiv_op = true;
       break;
     case OPERATOR_LSHIFT:
       code = LSHIFT_EXPR;
@@ -6313,6 +5734,7 @@ Binary_expression::do_get_tree(Translate_context* context)
       go_unreachable();
     }
 
+  location_t gccloc = this->location().gcc_location();
   tree type = use_left_type ? TREE_TYPE(left) : TREE_TYPE(right);
 
   if (this->left_->type()->is_string_type())
@@ -6340,27 +5762,27 @@ Binary_expression::do_get_tree(Translate_context* context)
     }
 
   tree eval_saved = NULL_TREE;
-  if (is_shift_op)
+  if (is_shift_op
+      || (is_idiv_op && (go_check_divide_zero || go_check_divide_overflow)))
     {
       // Make sure the values are evaluated.
-      if (!DECL_P(left) && TREE_SIDE_EFFECTS(left))
+      if (!DECL_P(left))
        {
          left = save_expr(left);
          eval_saved = left;
        }
-      if (!DECL_P(right) && TREE_SIDE_EFFECTS(right))
+      if (!DECL_P(right))
        {
          right = save_expr(right);
          if (eval_saved == NULL_TREE)
            eval_saved = right;
          else
-           eval_saved = fold_build2_loc(this->location(), COMPOUND_EXPR,
+           eval_saved = fold_build2_loc(gccloc, COMPOUND_EXPR,
                                         void_type_node, eval_saved, right);
        }
     }
 
-  tree ret = fold_build2_loc(this->location(),
-                            code,
+  tree ret = fold_build2_loc(gccloc, code,
                             compute_type != NULL_TREE ? compute_type : type,
                             left, right);
 
@@ -6378,50 +5800,130 @@ Binary_expression::do_get_tree(Translate_context* context)
       tree compare = fold_build2(LT_EXPR, boolean_type_node, right,
                                 build_int_cst_type(TREE_TYPE(right), bits));
 
-      tree overflow_result = fold_convert_loc(this->location(),
-                                             TREE_TYPE(left),
+      tree overflow_result = fold_convert_loc(gccloc, TREE_TYPE(left),
                                              integer_zero_node);
       if (this->op_ == OPERATOR_RSHIFT
          && !this->left_->type()->integer_type()->is_unsigned())
        {
-         tree neg = fold_build2_loc(this->location(), LT_EXPR,
-                                    boolean_type_node, left,
-                                    fold_convert_loc(this->location(),
-                                                     TREE_TYPE(left),
-                                                     integer_zero_node));
-         tree neg_one = fold_build2_loc(this->location(),
-                                        MINUS_EXPR, TREE_TYPE(left),
-                                        fold_convert_loc(this->location(),
-                                                         TREE_TYPE(left),
-                                                         integer_zero_node),
-                                        fold_convert_loc(this->location(),
-                                                         TREE_TYPE(left),
-                                                         integer_one_node));
-         overflow_result = fold_build3_loc(this->location(), COND_EXPR,
-                                           TREE_TYPE(left), neg, neg_one,
-                                           overflow_result);
-       }
-
-      ret = fold_build3_loc(this->location(), COND_EXPR, TREE_TYPE(left),
+         tree neg =
+            fold_build2_loc(gccloc, LT_EXPR, boolean_type_node,
+                           left,
+                            fold_convert_loc(gccloc, TREE_TYPE(left),
+                                             integer_zero_node));
+         tree neg_one =
+            fold_build2_loc(gccloc, MINUS_EXPR, TREE_TYPE(left),
+                            fold_convert_loc(gccloc, TREE_TYPE(left),
+                                             integer_zero_node),
+                            fold_convert_loc(gccloc, TREE_TYPE(left),
+                                             integer_one_node));
+         overflow_result =
+            fold_build3_loc(gccloc, COND_EXPR, TREE_TYPE(left),
+                           neg, neg_one, overflow_result);
+       }
+
+      ret = fold_build3_loc(gccloc, COND_EXPR, TREE_TYPE(left),
                            compare, ret, overflow_result);
 
       if (eval_saved != NULL_TREE)
-       ret = fold_build2_loc(this->location(), COMPOUND_EXPR,
-                             TREE_TYPE(ret), eval_saved, ret);
+       ret = fold_build2_loc(gccloc, COMPOUND_EXPR, TREE_TYPE(ret),
+                             eval_saved, ret);
     }
 
-  return ret;
-}
+  // Add checks for division by zero and division overflow as needed.
+  if (is_idiv_op)
+    {
+      if (go_check_divide_zero)
+       {
+         // right == 0
+         tree check = fold_build2_loc(gccloc, EQ_EXPR, boolean_type_node,
+                                      right,
+                                      fold_convert_loc(gccloc,
+                                                       TREE_TYPE(right),
+                                                       integer_zero_node));
 
-// Export a binary expression.
+         // __go_runtime_error(RUNTIME_ERROR_DIVISION_BY_ZERO), 0
+         int errcode = RUNTIME_ERROR_DIVISION_BY_ZERO;
+         tree panic = fold_build2_loc(gccloc, COMPOUND_EXPR, TREE_TYPE(ret),
+                                      Gogo::runtime_error(errcode,
+                                                          this->location()),
+                                      fold_convert_loc(gccloc, TREE_TYPE(ret),
+                                                       integer_zero_node));
 
-void
-Binary_expression::do_export(Export* exp) const
-{
-  exp->write_c_string("(");
-  this->left_->export_expression(exp);
-  switch (this->op_)
-    {
+         // right == 0 ? (__go_runtime_error(...), 0) : ret
+         ret = fold_build3_loc(gccloc, COND_EXPR, TREE_TYPE(ret),
+                               check, panic, ret);
+       }
+
+      if (go_check_divide_overflow)
+       {
+         // right == -1
+         // FIXME: It would be nice to say that this test is expected
+         // to return false.
+         tree m1 = integer_minus_one_node;
+         tree check = fold_build2_loc(gccloc, EQ_EXPR, boolean_type_node,
+                                      right,
+                                      fold_convert_loc(gccloc,
+                                                       TREE_TYPE(right),
+                                                       m1));
+
+         tree overflow;
+         if (TYPE_UNSIGNED(TREE_TYPE(ret)))
+           {
+             // An unsigned -1 is the largest possible number, so
+             // dividing is always 1 or 0.
+             tree cmp = fold_build2_loc(gccloc, EQ_EXPR, boolean_type_node,
+                                        left, right);
+             if (this->op_ == OPERATOR_DIV)
+               overflow = fold_build3_loc(gccloc, COND_EXPR, TREE_TYPE(ret),
+                                          cmp,
+                                          fold_convert_loc(gccloc,
+                                                           TREE_TYPE(ret),
+                                                           integer_one_node),
+                                          fold_convert_loc(gccloc,
+                                                           TREE_TYPE(ret),
+                                                           integer_zero_node));
+             else
+               overflow = fold_build3_loc(gccloc, COND_EXPR, TREE_TYPE(ret),
+                                          cmp,
+                                          fold_convert_loc(gccloc,
+                                                           TREE_TYPE(ret),
+                                                           integer_zero_node),
+                                          left);
+           }
+         else
+           {
+             // Computing left / -1 is the same as computing - left,
+             // which does not overflow since Go sets -fwrapv.
+             if (this->op_ == OPERATOR_DIV)
+               overflow = fold_build1_loc(gccloc, NEGATE_EXPR, TREE_TYPE(left),
+                                          left);
+             else
+               overflow = integer_zero_node;
+           }
+         overflow = fold_convert_loc(gccloc, TREE_TYPE(ret), overflow);
+
+         // right == -1 ? - left : ret
+         ret = fold_build3_loc(gccloc, COND_EXPR, TREE_TYPE(ret),
+                               check, overflow, ret);
+       }
+
+      if (eval_saved != NULL_TREE)
+       ret = fold_build2_loc(gccloc, COMPOUND_EXPR, TREE_TYPE(ret),
+                             eval_saved, ret);
+    }
+
+  return ret;
+}
+
+// Export a binary expression.
+
+void
+Binary_expression::do_export(Export* exp) const
+{
+  exp->write_c_string("(");
+  this->left_->export_expression(exp);
+  switch (this->op_)
+    {
     case OPERATOR_OROR:
       exp->write_c_string(" || ");
       break;
@@ -6622,7 +6124,7 @@ Binary_expression::do_dump_expression(Ast_dump_context* ast_dump_context) const
 
 Expression*
 Expression::make_binary(Operator op, Expression* left, Expression* right,
-                       source_location location)
+                       Location location)
 {
   return new Binary_expression(op, left, right, location);
 }
@@ -6633,7 +6135,7 @@ tree
 Expression::comparison_tree(Translate_context* context, Operator op,
                            Type* left_type, tree left_tree,
                            Type* right_type, tree right_tree,
-                           source_location location)
+                           Location location)
 {
   enum tree_code code;
   switch (op)
@@ -6699,10 +6201,12 @@ Expression::comparison_tree(Translate_context* context, Operator op,
          make_tmp = NULL_TREE;
          arg = right_tree;
        }
-      else if (TREE_ADDRESSABLE(TREE_TYPE(right_tree)) || DECL_P(right_tree))
+      else if (TREE_ADDRESSABLE(TREE_TYPE(right_tree))
+              || (TREE_CODE(right_tree) != CONST_DECL
+                  && DECL_P(right_tree)))
        {
          make_tmp = NULL_TREE;
-         arg = build_fold_addr_expr_loc(location, right_tree);
+         arg = build_fold_addr_expr_loc(location.gcc_location(), right_tree);
          if (DECL_P(right_tree))
            TREE_ADDRESSABLE(right_tree) = 1;
        }
@@ -6714,10 +6218,10 @@ Expression::comparison_tree(Translate_context* context, Operator op,
          DECL_INITIAL(tmp) = right_tree;
          TREE_ADDRESSABLE(tmp) = 1;
          make_tmp = build1(DECL_EXPR, void_type_node, tmp);
-         SET_EXPR_LOCATION(make_tmp, location);
-         arg = build_fold_addr_expr_loc(location, tmp);
+         SET_EXPR_LOCATION(make_tmp, location.gcc_location());
+         arg = build_fold_addr_expr_loc(location.gcc_location(), tmp);
        }
-      arg = fold_convert_loc(location, ptr_type_node, arg);
+      arg = fold_convert_loc(location.gcc_location(), ptr_type_node, arg);
 
       tree descriptor = right_type->type_descriptor_pointer(context->gogo(),
                                                            location);
@@ -6872,7 +6376,7 @@ Expression::comparison_tree(Translate_context* context, Operator op,
 
   tree ret = fold_build2(code, boolean_type_node, left_tree, right_tree);
   if (CAN_HAVE_LOCATION_P(ret))
-    SET_EXPR_LOCATION(ret, location);
+    SET_EXPR_LOCATION(ret, location.gcc_location());
   return ret;
 }
 
@@ -6972,7 +6476,7 @@ Bound_method_expression::do_dump_expression(Ast_dump_context* ast_dump_context)
 
 Bound_method_expression*
 Expression::make_bound_method(Expression* expr, Named_object* method,
-                             source_location location)
+                             Location location)
 {
   return new Bound_method_expression(expr, method, location);
 }
@@ -6984,7 +6488,7 @@ class Builtin_call_expression : public Call_expression
 {
  public:
   Builtin_call_expression(Gogo* gogo, Expression* fn, Expression_list* args,
-                         bool is_varargs, source_location location);
+                         bool is_varargs, Location location);
 
  protected:
   // This overrides Call_expression::do_lower.
@@ -6995,13 +6499,7 @@ class Builtin_call_expression : public Call_expression
   do_is_constant() const;
 
   bool
-  do_integer_constant_value(bool, mpz_t, Type**) const;
-
-  bool
-  do_float_constant_value(mpfr_t, Type**) const;
-
-  bool
-  do_complex_constant_value(mpfr_t, mpfr_t, Type**) const;
+  do_numeric_constant_value(Numeric_constant*) const;
 
   void
   do_discarding_value();
@@ -7097,7 +6595,7 @@ Builtin_call_expression::Builtin_call_expression(Gogo* gogo,
                                                 Expression* fn,
                                                 Expression_list* args,
                                                 bool is_varargs,
-                                                source_location location)
+                                                Location location)
   : Call_expression(fn, args, is_varargs, location),
     gogo_(gogo), code_(BUILTIN_INVALID), seen_(false)
 {
@@ -7209,7 +6707,7 @@ Builtin_call_expression::do_lower(Gogo* gogo, Named_object* function,
   if (this->classification() == EXPRESSION_ERROR)
     return this;
 
-  source_location loc = this->location();
+  Location loc = this->location();
 
   if (this->is_varargs() && this->code_ != BUILTIN_APPEND)
     {
@@ -7224,7 +6722,7 @@ Builtin_call_expression::do_lower(Gogo* gogo, Named_object* function,
       if (this->code_ == BUILTIN_LEN || this->code_ == BUILTIN_CAP)
        {
          Expression* arg = this->one_arg();
-         if (!arg->is_constant())
+         if (arg != NULL && !arg->is_constant())
            {
              Find_call_expression find_call;
              Expression::traverse(&arg, &find_call);
@@ -7233,37 +6731,9 @@ Builtin_call_expression::do_lower(Gogo* gogo, Named_object* function,
            }
        }
 
-      mpz_t ival;
-      mpz_init(ival);
-      Type* type;
-      if (this->integer_constant_value(true, ival, &type))
-       {
-         Expression* ret = Expression::make_integer(&ival, type, loc);
-         mpz_clear(ival);
-         return ret;
-       }
-      mpz_clear(ival);
-
-      mpfr_t rval;
-      mpfr_init(rval);
-      if (this->float_constant_value(rval, &type))
-       {
-         Expression* ret = Expression::make_float(&rval, type, loc);
-         mpfr_clear(rval);
-         return ret;
-       }
-
-      mpfr_t imag;
-      mpfr_init(imag);
-      if (this->complex_constant_value(rval, imag, &type))
-       {
-         Expression* ret = Expression::make_complex(&rval, &imag, type, loc);
-         mpfr_clear(rval);
-         mpfr_clear(imag);
-         return ret;
-       }
-      mpfr_clear(rval);
-      mpfr_clear(imag);
+      Numeric_constant nc;
+      if (this->numeric_constant_value(&nc))
+       return nc.expression(loc);
     }
 
   switch (this->code_)
@@ -7302,7 +6772,7 @@ Builtin_call_expression::do_lower(Gogo* gogo, Named_object* function,
        {
          // Calling recover outside of a function always returns the
          // nil empty interface.
-         Type* eface = Type::make_interface_type(NULL, loc);
+         Type* eface = Type::make_empty_interface_type(loc);
          return Expression::make_cast(eface, Expression::make_nil(loc), loc);
        }
       break;
@@ -7320,7 +6790,10 @@ Builtin_call_expression::do_lower(Gogo* gogo, Named_object* function,
            this->set_is_error();
            return this;
          }
-       this->lower_varargs(gogo, function, inserter, slice_type, 2);
+       Type* element_type = slice_type->array_type()->element_type();
+       this->lower_varargs(gogo, function, inserter,
+                           Type::make_array_type(element_type, NULL),
+                           2);
       }
       break;
 
@@ -7369,7 +6842,7 @@ Builtin_call_expression::do_lower(Gogo* gogo, Named_object* function,
 Expression*
 Builtin_call_expression::lower_make()
 {
-  source_location loc = this->location();
+  Location loc = this->location();
 
   const Expression_list* args = this->args();
   if (args == NULL || args->size() < 1)
@@ -7404,6 +6877,10 @@ Builtin_call_expression::lower_make()
       return Expression::make_error(this->location());
     }
 
+  bool have_big_args = false;
+  Type* uintptr_type = Type::lookup_integer_type("uintptr");
+  int uintptr_bits = uintptr_type->integer_type()->bits();
+
   ++parg;
   Expression* len_arg;
   if (parg == args->end())
@@ -7427,6 +6904,9 @@ Builtin_call_expression::lower_make()
          this->report_error(_("bad size for make"));
          return Expression::make_error(this->location());
        }
+      if (len_arg->type()->integer_type() != NULL
+         && len_arg->type()->integer_type()->bits() > uintptr_bits)
+       have_big_args = true;
       ++parg;
     }
 
@@ -7439,6 +6919,9 @@ Builtin_call_expression::lower_make()
          this->report_error(_("bad capacity when making slice"));
          return Expression::make_error(this->location());
        }
+      if (cap_arg->type()->integer_type() != NULL
+         && cap_arg->type()->integer_type()->bits() > uintptr_bits)
+       have_big_args = true;
       ++parg;
     }
 
@@ -7448,7 +6931,7 @@ Builtin_call_expression::lower_make()
       return Expression::make_error(this->location());
     }
 
-  source_location type_loc = first_arg->location();
+  Location type_loc = first_arg->location();
   Expression* type_arg;
   if (is_slice || is_chan)
     type_arg = Expression::make_type_descriptor(type, type_loc);
@@ -7461,16 +6944,26 @@ Builtin_call_expression::lower_make()
   if (is_slice)
     {
       if (cap_arg == NULL)
-       call = Runtime::make_call(Runtime::MAKESLICE1, loc, 2, type_arg,
-                                 len_arg);
+       call = Runtime::make_call((have_big_args
+                                  ? Runtime::MAKESLICE1BIG
+                                  : Runtime::MAKESLICE1),
+                                 loc, 2, type_arg, len_arg);
       else
-       call = Runtime::make_call(Runtime::MAKESLICE2, loc, 3, type_arg,
-                                 len_arg, cap_arg);
+       call = Runtime::make_call((have_big_args
+                                  ? Runtime::MAKESLICE2BIG
+                                  : Runtime::MAKESLICE2),
+                                 loc, 3, type_arg, len_arg, cap_arg);
     }
   else if (is_map)
-    call = Runtime::make_call(Runtime::MAKEMAP, loc, 2, type_arg, len_arg);
+    call = Runtime::make_call((have_big_args
+                              ? Runtime::MAKEMAPBIG
+                              : Runtime::MAKEMAP),
+                             loc, 2, type_arg, len_arg);
   else if (is_chan)
-    call = Runtime::make_call(Runtime::MAKECHAN, loc, 2, type_arg, len_arg);
+    call = Runtime::make_call((have_big_args
+                              ? Runtime::MAKECHANBIG
+                              : Runtime::MAKECHAN),
+                             loc, 2, type_arg, len_arg);
   else
     go_unreachable();
 
@@ -7488,43 +6981,14 @@ Builtin_call_expression::check_int_value(Expression* e)
     return true;
 
   // Check for a floating point constant with integer value.
-  mpfr_t fval;
-  mpfr_init(fval);
-
-  Type* dummy;
-  if (e->float_constant_value(fval, &dummy) && mpfr_integer_p(fval))
+  Numeric_constant nc;
+  mpz_t ival;
+  if (e->numeric_constant_value(&nc) && nc.to_int(&ival))
     {
-      mpz_t ival;
-      mpz_init(ival);
-
-      bool ok = false;
-
-      mpfr_clear_overflow();
-      mpfr_clear_erangeflag();
-      mpfr_get_z(ival, fval, GMP_RNDN);
-      if (!mpfr_overflow_p()
-         && !mpfr_erangeflag_p()
-         && mpz_sgn(ival) >= 0)
-       {
-         Named_type* ntype = Type::lookup_integer_type("int");
-         Integer_type* inttype = ntype->integer_type();
-         mpz_t max;
-         mpz_init_set_ui(max, 1);
-         mpz_mul_2exp(max, max, inttype->bits() - 1);
-         ok = mpz_cmp(ival, max) < 0;
-         mpz_clear(max);
-       }
       mpz_clear(ival);
-
-      if (ok)
-       {
-         mpfr_clear(fval);
-         return true;
-       }
+      return true;
     }
 
-  mpfr_clear(fval);
-
   return false;
 }
 
@@ -7578,7 +7042,7 @@ Expression*
 Builtin_call_expression::one_arg() const
 {
   const Expression_list* args = this->args();
-  if (args->size() != 1)
+  if (args == NULL || args->size() != 1)
     return NULL;
   return args->front();
 }
@@ -7655,12 +7119,10 @@ Builtin_call_expression::do_is_constant() const
   return false;
 }
 
-// Return an integer constant value if possible.
+// Return a numeric constant if possible.
 
 bool
-Builtin_call_expression::do_integer_constant_value(bool iota_is_constant,
-                                                  mpz_t val,
-                                                  Type** ptype) const
+Builtin_call_expression::do_numeric_constant_value(Numeric_constant* nc) const
 {
   if (this->code_ == BUILTIN_LEN
       || this->code_ == BUILTIN_CAP)
@@ -7675,8 +7137,8 @@ Builtin_call_expression::do_integer_constant_value(bool iota_is_constant,
          std::string sval;
          if (arg->string_constant_value(&sval))
            {
-             mpz_set_ui(val, sval.length());
-             *ptype = Type::lookup_integer_type("int");
+             nc->set_unsigned_long(Type::lookup_integer_type("int"),
+                                   sval.length());
              return true;
            }
        }
@@ -7693,13 +7155,15 @@ Builtin_call_expression::do_integer_constant_value(bool iota_is_constant,
            return false;
          Expression* e = arg_type->array_type()->length();
          this->seen_ = true;
-         bool r = e->integer_constant_value(iota_is_constant, val, ptype);
+         bool r = e->numeric_constant_value(nc);
          this->seen_ = false;
          if (r)
            {
-             *ptype = Type::lookup_integer_type("int");
-             return true;
+             if (!nc->set_type(Type::lookup_integer_type("int"), false,
+                               this->location()))
+               r = false;
            }
+         return r;
        }
     }
   else if (this->code_ == BUILTIN_SIZEOF
@@ -7715,36 +7179,33 @@ Builtin_call_expression::do_integer_constant_value(bool iota_is_constant,
        return false;
       if (arg_type->named_type() != NULL)
        arg_type->named_type()->convert(this->gogo_);
-      tree arg_type_tree = type_to_tree(arg_type->get_backend(this->gogo_));
-      if (arg_type_tree == error_mark_node)
-       return false;
-      unsigned long val_long;
+
+      unsigned int ret;
       if (this->code_ == BUILTIN_SIZEOF)
        {
-         tree type_size = TYPE_SIZE_UNIT(arg_type_tree);
-         go_assert(TREE_CODE(type_size) == INTEGER_CST);
-         if (TREE_INT_CST_HIGH(type_size) != 0)
-           return false;
-         unsigned HOST_WIDE_INT val_wide = TREE_INT_CST_LOW(type_size);
-         val_long = static_cast<unsigned long>(val_wide);
-         if (val_long != val_wide)
+         if (!arg_type->backend_type_size(this->gogo_, &ret))
            return false;
        }
       else if (this->code_ == BUILTIN_ALIGNOF)
        {
          if (arg->field_reference_expression() == NULL)
-           val_long = go_type_alignment(arg_type_tree);
+           {
+             if (!arg_type->backend_type_align(this->gogo_, &ret))
+               return false;
+           }
          else
            {
              // Calling unsafe.Alignof(s.f) returns the alignment of
              // the type of f when it is used as a field in a struct.
-             val_long = go_field_alignment(arg_type_tree);
+             if (!arg_type->backend_type_field_align(this->gogo_, &ret))
+               return false;
            }
        }
       else
        go_unreachable();
-      mpz_set_ui(val, val_long);
-      *ptype = NULL;
+
+      nc->set_unsigned_long(Type::lookup_integer_type("uintptr"),
+                           static_cast<unsigned long>(ret));
       return true;
     }
   else if (this->code_ == BUILTIN_OFFSETOF)
@@ -7761,102 +7222,78 @@ Builtin_call_expression::do_integer_constant_value(bool iota_is_constant,
        return false;
       if (st->named_type() != NULL)
        st->named_type()->convert(this->gogo_);
-      tree struct_tree = type_to_tree(st->get_backend(this->gogo_));
-      go_assert(TREE_CODE(struct_tree) == RECORD_TYPE);
-      tree field = TYPE_FIELDS(struct_tree);
-      for (unsigned int index = farg->field_index(); index > 0; --index)
-       {
-         field = DECL_CHAIN(field);
-         go_assert(field != NULL_TREE);
-       }
-      HOST_WIDE_INT offset_wide = int_byte_position (field);
-      if (offset_wide < 0)
-       return false;
-      unsigned long offset_long = static_cast<unsigned long>(offset_wide);
-      if (offset_long != static_cast<unsigned HOST_WIDE_INT>(offset_wide))
+      unsigned int offset;
+      if (!st->struct_type()->backend_field_offset(this->gogo_,
+                                                  farg->field_index(),
+                                                  &offset))
        return false;
-      mpz_set_ui(val, offset_long);
+      nc->set_unsigned_long(Type::lookup_integer_type("uintptr"),
+                           static_cast<unsigned long>(offset));
       return true;
     }
-  return false;
-}
-
-// Return a floating point constant value if possible.
-
-bool
-Builtin_call_expression::do_float_constant_value(mpfr_t val,
-                                                Type** ptype) const
-{
-  if (this->code_ == BUILTIN_REAL || this->code_ == BUILTIN_IMAG)
+  else if (this->code_ == BUILTIN_REAL || this->code_ == BUILTIN_IMAG)
     {
       Expression* arg = this->one_arg();
       if (arg == NULL)
        return false;
 
+      Numeric_constant argnc;
+      if (!arg->numeric_constant_value(&argnc))
+       return false;
+
       mpfr_t real;
       mpfr_t imag;
-      mpfr_init(real);
-      mpfr_init(imag);
-
-      bool ret = false;
-      Type* type;
-      if (arg->complex_constant_value(real, imag, &type))
-       {
-         if (this->code_ == BUILTIN_REAL)
-           mpfr_set(val, real, GMP_RNDN);
-         else
-           mpfr_set(val, imag, GMP_RNDN);
-         *ptype = Builtin_call_expression::real_imag_type(type);
-         ret = true;
-       }
+      if (!argnc.to_complex(&real, &imag))
+       return false;
 
-      mpfr_clear(real);
-      mpfr_clear(imag);
-      return ret;
+      Type* type = Builtin_call_expression::real_imag_type(argnc.type());
+      if (this->code_ == BUILTIN_REAL)
+       nc->set_float(type, real);
+      else
+       nc->set_float(type, imag);
+      return true;
     }
-
-  return false;
-}
-
-// Return a complex constant value if possible.
-
-bool
-Builtin_call_expression::do_complex_constant_value(mpfr_t real, mpfr_t imag,
-                                                  Type** ptype) const
-{
-  if (this->code_ == BUILTIN_COMPLEX)
+  else if (this->code_ == BUILTIN_COMPLEX)
     {
       const Expression_list* args = this->args();
       if (args == NULL || args->size() != 2)
        return false;
 
+      Numeric_constant rnc;
+      if (!args->front()->numeric_constant_value(&rnc))
+       return false;
+      Numeric_constant inc;
+      if (!args->back()->numeric_constant_value(&inc))
+       return false;
+
+      if (rnc.type() != NULL
+         && !rnc.type()->is_abstract()
+         && inc.type() != NULL
+         && !inc.type()->is_abstract()
+         && !Type::are_identical(rnc.type(), inc.type(), false, NULL))
+       return false;
+
       mpfr_t r;
-      mpfr_init(r);
-      Type* rtype;
-      if (!args->front()->float_constant_value(r, &rtype))
+      if (!rnc.to_float(&r))
+       return false;
+      mpfr_t i;
+      if (!inc.to_float(&i))
        {
          mpfr_clear(r);
          return false;
        }
 
-      mpfr_t i;
-      mpfr_init(i);
+      Type* arg_type = rnc.type();
+      if (arg_type == NULL || arg_type->is_abstract())
+       arg_type = inc.type();
 
-      bool ret = false;
-      Type* itype;
-      if (args->back()->float_constant_value(i, &itype)
-         && Type::are_identical(rtype, itype, false, NULL))
-       {
-         mpfr_set(real, r, GMP_RNDN);
-         mpfr_set(imag, i, GMP_RNDN);
-         *ptype = Builtin_call_expression::complex_type(rtype);
-         ret = true;
-       }
+      Type* type = Builtin_call_expression::complex_type(arg_type);
+      nc->set_complex(type, r, i);
 
       mpfr_clear(r);
       mpfr_clear(i);
 
-      return ret;
+      return true;
     }
 
   return false;
@@ -7924,10 +7361,12 @@ Builtin_call_expression::do_type()
     case BUILTIN_CAP:
     case BUILTIN_COPY:
     case BUILTIN_LEN:
+      return Type::lookup_integer_type("int");
+
     case BUILTIN_ALIGNOF:
     case BUILTIN_OFFSETOF:
     case BUILTIN_SIZEOF:
-      return Type::lookup_integer_type("int");
+      return Type::lookup_integer_type("uintptr");
 
     case BUILTIN_CLOSE:
     case BUILTIN_DELETE:
@@ -7937,7 +7376,7 @@ Builtin_call_expression::do_type()
       return Type::make_void_type();
 
     case BUILTIN_RECOVER:
-      return Type::make_interface_type(NULL, BUILTINS_LOCATION);
+      return Type::make_empty_interface_type(Linemap::predeclared_location());
 
     case BUILTIN_APPEND:
       {
@@ -8055,15 +7494,19 @@ Builtin_call_expression::do_determine_type(const Type_context* context)
                {
                  if (atype->integer_type() != NULL)
                    {
-                     mpz_t val;
-                     mpz_init(val);
-                     Type* dummy;
-                     if (this->integer_constant_value(true, val, &dummy)
-                         && mpz_sgn(val) >= 0)
-                       want_type = Type::lookup_integer_type("uint64");
-                     else
+                     Numeric_constant nc;
+                     if (this->numeric_constant_value(&nc))
+                       {
+                         mpz_t val;
+                         if (nc.to_int(&val))
+                           {
+                             if (mpz_sgn(val) >= 0)
+                               want_type = Type::lookup_integer_type("uint64");
+                             mpz_clear(val);
+                           }
+                       }
+                     if (want_type == NULL)
                        want_type = Type::lookup_integer_type("int64");
-                     mpz_clear(val);
                    }
                  else if (atype->float_type() != NULL)
                    want_type = Type::lookup_float_type("float64");
@@ -8115,11 +7558,14 @@ Builtin_call_expression::check_one_arg()
 void
 Builtin_call_expression::do_check_types(Gogo*)
 {
+  if (this->is_error_expression())
+    return;
   switch (this->code_)
     {
     case BUILTIN_INVALID:
     case BUILTIN_NEW:
     case BUILTIN_MAKE:
+    case BUILTIN_DELETE:
       return;
 
     case BUILTIN_LEN:
@@ -8189,6 +7635,11 @@ Builtin_call_expression::do_check_types(Gogo*)
                    || type->function_type() != NULL
                    || type->is_slice_type())
                  ;
+               else if ((*p)->is_type_expression())
+                 {
+                   // If this is a type expression it's going to give
+                   // an error anyhow, so we don't need one here.
+                 }
                else
                  this->report_error(_("unsupported argument type to "
                                       "builtin function"));
@@ -8254,19 +7705,19 @@ Builtin_call_expression::do_check_types(Gogo*)
            break;
          }
 
-       Type* e2;
        if (arg2_type->is_slice_type())
-         e2 = arg2_type->array_type()->element_type();
+         {
+           Type* e2 = arg2_type->array_type()->element_type();
+           if (!Type::are_identical(e1, e2, true, NULL))
+             this->report_error(_("element types must be the same"));
+         }
        else if (arg2_type->is_string_type())
-         e2 = Type::lookup_integer_type("uint8");
-       else
          {
-           this->report_error(_("right argument must be a slice or a string"));
-           break;
+           if (e1->integer_type() == NULL || !e1->integer_type()->is_byte())
+             this->report_error(_("first argument must be []byte"));
          }
-
-       if (!Type::are_identical(e1, e2, true, NULL))
-         this->report_error(_("element types must be the same"));
+       else
+           this->report_error(_("second argument must be slice or string"));
       }
       break;
 
@@ -8283,27 +7734,34 @@ Builtin_call_expression::do_check_types(Gogo*)
            this->report_error(_("too many arguments"));
            break;
          }
+       if (args->front()->type()->is_error()
+           || args->back()->type()->is_error())
+         break;
+
+       Array_type* at = args->front()->type()->array_type();
+       Type* e = at->element_type();
 
        // The language permits appending a string to a []byte, as a
        // special case.
        if (args->back()->type()->is_string_type())
          {
-           const Array_type* at = args->front()->type()->array_type();
-           const Type* e = at->element_type()->forwarded();
-           if (e == Type::lookup_integer_type("uint8"))
+           if (e->integer_type() != NULL && e->integer_type()->is_byte())
              break;
          }
 
+       // The language says that the second argument must be
+       // assignable to a slice of the element type of the first
+       // argument.  We already know the first argument is a slice
+       // type.
+       Type* arg2_type = Type::make_array_type(e, NULL);
        std::string reason;
-       if (!Type::are_assignable(args->front()->type(), args->back()->type(),
-                                 &reason))
+       if (!Type::are_assignable(arg2_type, args->back()->type(), &reason))
          {
            if (reason.empty())
-             this->report_error(_("arguments 1 and 2 have different types"));
+             this->report_error(_("argument 2 has invalid type"));
            else
              {
-               error_at(this->location(),
-                        "arguments 1 and 2 have different types (%s)",
+               error_at(this->location(), "argument 2 has invalid type (%s)",
                         reason.c_str());
                this->set_is_error();
              }
@@ -8352,7 +7810,7 @@ tree
 Builtin_call_expression::do_get_tree(Translate_context* context)
 {
   Gogo* gogo = context->gogo();
-  source_location location = this->location();
+  Location location = this->location();
   switch (this->code_)
     {
     case BUILTIN_INVALID:
@@ -8523,7 +7981,8 @@ Builtin_call_expression::do_get_tree(Translate_context* context)
                    fnname = "__go_print_uint64";
                    Type* itype = Type::lookup_integer_type("uint64");
                    Btype* bitype = itype->get_backend(gogo);
-                   arg = fold_convert_loc(location, type_to_tree(bitype), arg);
+                   arg = fold_convert_loc(location.gcc_location(),
+                                           type_to_tree(bitype), arg);
                  }
                else if (type->integer_type() != NULL)
                  {
@@ -8532,22 +7991,24 @@ Builtin_call_expression::do_get_tree(Translate_context* context)
                    fnname = "__go_print_int64";
                    Type* itype = Type::lookup_integer_type("int64");
                    Btype* bitype = itype->get_backend(gogo);
-                   arg = fold_convert_loc(location, type_to_tree(bitype), arg);
+                   arg = fold_convert_loc(location.gcc_location(),
+                                           type_to_tree(bitype), arg);
                  }
                else if (type->float_type() != NULL)
                  {
                    static tree print_double_fndecl;
                    pfndecl = &print_double_fndecl;
                    fnname = "__go_print_double";
-                   arg = fold_convert_loc(location, double_type_node, arg);
+                   arg = fold_convert_loc(location.gcc_location(),
+                                           double_type_node, arg);
                  }
                else if (type->complex_type() != NULL)
                  {
                    static tree print_complex_fndecl;
                    pfndecl = &print_complex_fndecl;
                    fnname = "__go_print_complex";
-                   arg = fold_convert_loc(location, complex_double_type_node,
-                                          arg);
+                   arg = fold_convert_loc(location.gcc_location(),
+                                           complex_double_type_node, arg);
                  }
                else if (type->is_boolean_type())
                  {
@@ -8563,7 +8024,8 @@ Builtin_call_expression::do_get_tree(Translate_context* context)
                    static tree print_pointer_fndecl;
                    pfndecl = &print_pointer_fndecl;
                    fnname = "__go_print_pointer";
-                   arg = fold_convert_loc(location, ptr_type_node, arg);
+                   arg = fold_convert_loc(location.gcc_location(),
+                                           ptr_type_node, arg);
                  }
                else if (type->interface_type() != NULL)
                  {
@@ -8587,7 +8049,10 @@ Builtin_call_expression::do_get_tree(Translate_context* context)
                    fnname = "__go_print_slice";
                  }
                else
-                 go_unreachable();
+                 {
+                   go_assert(saw_errors());
+                   return error_mark_node;
+                 }
 
                tree call = Gogo::call_builtin(pfndecl,
                                               location,
@@ -8626,7 +8091,8 @@ Builtin_call_expression::do_get_tree(Translate_context* context)
        tree arg_tree = arg->get_tree(context);
        if (arg_tree == error_mark_node)
          return error_mark_node;
-       Type *empty = Type::make_interface_type(NULL, BUILTINS_LOCATION);
+       Type *empty =
+         Type::make_empty_interface_type(Linemap::predeclared_location());
        arg_tree = Expression::convert_for_assignment(context, empty,
                                                      arg->type(),
                                                      arg_tree, location);
@@ -8658,7 +8124,8 @@ Builtin_call_expression::do_get_tree(Translate_context* context)
        if (arg_tree == error_mark_node)
          return error_mark_node;
 
-       Type *empty = Type::make_interface_type(NULL, BUILTINS_LOCATION);
+       Type *empty =
+         Type::make_empty_interface_type(Linemap::predeclared_location());
        tree empty_tree = type_to_tree(empty->get_backend(context->gogo()));
 
        Type* nil_type = Type::make_nil_type();
@@ -8694,8 +8161,8 @@ Builtin_call_expression::do_get_tree(Translate_context* context)
          }
        if (call == error_mark_node)
          return error_mark_node;
-       return fold_build3_loc(location, COND_EXPR, empty_tree, arg_tree,
-                              call, empty_nil_tree);
+       return fold_build3_loc(location.gcc_location(), COND_EXPR, empty_tree,
+                               arg_tree, call, empty_nil_tree);
       }
 
     case BUILTIN_CLOSE:
@@ -8720,20 +8187,17 @@ Builtin_call_expression::do_get_tree(Translate_context* context)
     case BUILTIN_OFFSETOF:
     case BUILTIN_ALIGNOF:
       {
-       mpz_t val;
-       mpz_init(val);
-       Type* dummy;
-       bool b = this->integer_constant_value(true, val, &dummy);
-       if (!b)
+       Numeric_constant nc;
+       unsigned long val;
+       if (!this->numeric_constant_value(&nc)
+           || nc.to_unsigned_long(&val) != Numeric_constant::NC_UL_VALID)
          {
            go_assert(saw_errors());
            return error_mark_node;
          }
-       Type* int_type = Type::lookup_integer_type("int");
-       tree type = type_to_tree(int_type->get_backend(gogo));
-       tree ret = Expression::integer_constant_tree(val, type);
-       mpz_clear(val);
-       return ret;
+       Type* uintptr_type = Type::lookup_integer_type("uintptr");
+       tree type = type_to_tree(uintptr_type->get_backend(gogo));
+       return build_int_cst(type, val);
       }
 
     case BUILTIN_COPY:
@@ -8777,9 +8241,10 @@ Builtin_call_expression::do_get_tree(Translate_context* context)
 
        arg1_len = save_expr(arg1_len);
        arg2_len = save_expr(arg2_len);
-       tree len = fold_build3_loc(location, COND_EXPR, TREE_TYPE(arg1_len),
-                                  fold_build2_loc(location, LT_EXPR,
-                                                  boolean_type_node,
+       tree len = fold_build3_loc(location.gcc_location(), COND_EXPR,
+                                   TREE_TYPE(arg1_len),
+                                  fold_build2_loc(location.gcc_location(),
+                                                   LT_EXPR, boolean_type_node,
                                                   arg1_len, arg2_len),
                                   arg1_len, arg2_len);
        len = save_expr(len);
@@ -8790,15 +8255,18 @@ Builtin_call_expression::do_get_tree(Translate_context* context)
        if (element_type_tree == error_mark_node)
          return error_mark_node;
        tree element_size = TYPE_SIZE_UNIT(element_type_tree);
-       tree bytecount = fold_convert_loc(location, TREE_TYPE(element_size),
-                                         len);
-       bytecount = fold_build2_loc(location, MULT_EXPR,
+       tree bytecount = fold_convert_loc(location.gcc_location(),
+                                          TREE_TYPE(element_size), len);
+       bytecount = fold_build2_loc(location.gcc_location(), MULT_EXPR,
                                    TREE_TYPE(element_size),
                                    bytecount, element_size);
-       bytecount = fold_convert_loc(location, size_type_node, bytecount);
+       bytecount = fold_convert_loc(location.gcc_location(), size_type_node,
+                                     bytecount);
 
-       arg1_val = fold_convert_loc(location, ptr_type_node, arg1_val);
-       arg2_val = fold_convert_loc(location, ptr_type_node, arg2_val);
+       arg1_val = fold_convert_loc(location.gcc_location(), ptr_type_node,
+                                    arg1_val);
+       arg2_val = fold_convert_loc(location.gcc_location(), ptr_type_node,
+                                    arg2_val);
 
        static tree copy_fndecl;
        tree call = Gogo::call_builtin(&copy_fndecl,
@@ -8815,8 +8283,8 @@ Builtin_call_expression::do_get_tree(Translate_context* context)
        if (call == error_mark_node)
          return error_mark_node;
 
-       return fold_build2_loc(location, COMPOUND_EXPR, TREE_TYPE(len),
-                              call, len);
+       return fold_build2_loc(location.gcc_location(), COMPOUND_EXPR,
+                               TREE_TYPE(len), call, len);
       }
 
     case BUILTIN_APPEND:
@@ -8838,7 +8306,8 @@ Builtin_call_expression::do_get_tree(Translate_context* context)
        tree arg2_len;
        tree element_size;
        if (arg2->type()->is_string_type()
-           && element_type == Type::lookup_integer_type("uint8"))
+           && element_type->integer_type() != NULL
+           && element_type->integer_type()->is_byte())
          {
            arg2_tree = save_expr(arg2_tree);
            arg2_val = String_type::bytes_tree(gogo, arg2_tree);
@@ -8866,9 +8335,11 @@ Builtin_call_expression::do_get_tree(Translate_context* context)
             element_size = TYPE_SIZE_UNIT(element_type_tree);
          }
 
-       arg2_val = fold_convert_loc(location, ptr_type_node, arg2_val);
-       arg2_len = fold_convert_loc(location, size_type_node, arg2_len);
-       element_size = fold_convert_loc(location, size_type_node,
+       arg2_val = fold_convert_loc(location.gcc_location(), ptr_type_node,
+                                    arg2_val);
+       arg2_len = fold_convert_loc(location.gcc_location(), size_type_node,
+                                    arg2_len);
+       element_size = fold_convert_loc(location.gcc_location(), size_type_node,
                                        element_size);
 
        if (arg2_val == error_mark_node
@@ -8905,11 +8376,11 @@ Builtin_call_expression::do_get_tree(Translate_context* context)
          return error_mark_node;
        go_assert(COMPLEX_FLOAT_TYPE_P(TREE_TYPE(arg_tree)));
        if (this->code_ == BUILTIN_REAL)
-         return fold_build1_loc(location, REALPART_EXPR,
+         return fold_build1_loc(location.gcc_location(), REALPART_EXPR,
                                 TREE_TYPE(TREE_TYPE(arg_tree)),
                                 arg_tree);
        else
-         return fold_build1_loc(location, IMAGPART_EXPR,
+         return fold_build1_loc(location.gcc_location(), IMAGPART_EXPR,
                                 TREE_TYPE(TREE_TYPE(arg_tree)),
                                 arg_tree);
       }
@@ -8925,7 +8396,7 @@ Builtin_call_expression::do_get_tree(Translate_context* context)
        go_assert(TYPE_MAIN_VARIANT(TREE_TYPE(r))
                   == TYPE_MAIN_VARIANT(TREE_TYPE(i)));
        go_assert(SCALAR_FLOAT_TYPE_P(TREE_TYPE(r)));
-       return fold_build2_loc(location, COMPLEX_EXPR,
+       return fold_build2_loc(location.gcc_location(), COMPLEX_EXPR,
                               build_complex_type(TREE_TYPE(r)),
                               r, i);
       }
@@ -8941,50 +8412,37 @@ Builtin_call_expression::do_get_tree(Translate_context* context)
 void
 Builtin_call_expression::do_export(Export* exp) const
 {
-  bool ok = false;
+  Numeric_constant nc;
+  if (!this->numeric_constant_value(&nc))
+    {
+      error_at(this->location(), "value is not constant");
+      return;
+    }
 
-  mpz_t val;
-  mpz_init(val);
-  Type* dummy;
-  if (this->integer_constant_value(true, val, &dummy))
+  if (nc.is_int())
     {
+      mpz_t val;
+      nc.get_int(&val);
       Integer_expression::export_integer(exp, val);
-      ok = true;
+      mpz_clear(val);
     }
-  mpz_clear(val);
-
-  if (!ok)
+  else if (nc.is_float())
     {
       mpfr_t fval;
-      mpfr_init(fval);
-      if (this->float_constant_value(fval, &dummy))
-       {
-         Float_expression::export_float(exp, fval);
-         ok = true;
-       }
+      nc.get_float(&fval);
+      Float_expression::export_float(exp, fval);
       mpfr_clear(fval);
     }
-
-  if (!ok)
+  else if (nc.is_complex())
     {
       mpfr_t real;
       mpfr_t imag;
-      mpfr_init(real);
-      mpfr_init(imag);
-      if (this->complex_constant_value(real, imag, &dummy))
-       {
-         Complex_expression::export_complex(exp, real, imag);
-         ok = true;
-       }
+      Complex_expression::export_complex(exp, real, imag);
       mpfr_clear(real);
       mpfr_clear(imag);
     }
-
-  if (!ok)
-    {
-      error_at(this->location(), "value is not constant");
-      return;
-    }
+  else
+    go_unreachable();
 
   // A trailing space lets us reliably identify the end of the number.
   exp->write_c_string(" ");
@@ -9013,7 +8471,7 @@ Expression*
 Call_expression::do_lower(Gogo* gogo, Named_object* function,
                          Statement_inserter* inserter, int)
 {
-  source_location loc = this->location();
+  Location loc = this->location();
 
   // A type cast can look like a function call.
   if (this->fn_->is_type_expression()
@@ -9160,7 +8618,7 @@ Call_expression::lower_varargs(Gogo* gogo, Named_object* function,
   if (this->varargs_are_lowered_)
     return;
 
-  source_location loc = this->location();
+  Location loc = this->location();
 
   go_assert(param_count > 0);
   go_assert(varargs_type->is_slice_type());
@@ -9200,7 +8658,14 @@ Call_expression::lower_varargs(Gogo* gogo, Named_object* function,
        new_args->push_back(*pa);
       else if (this->is_varargs_)
        {
-         this->report_error(_("too many arguments"));
+         if ((*pa)->type()->is_slice_type())
+           this->report_error(_("too many arguments"));
+         else
+           {
+             error_at(this->location(),
+                      _("invalid use of %<...%> with non-slice"));
+             this->set_is_error();
+           }
          return;
        }
       else
@@ -9211,7 +8676,7 @@ Call_expression::lower_varargs(Gogo* gogo, Named_object* function,
            {
              // Check types here so that we get a better message.
              Type* patype = (*pa)->type();
-             source_location paloc = (*pa)->location();
+             Location paloc = (*pa)->location();
              if (!this->check_argument_type(i, element_type, patype,
                                             paloc, issued_error))
                continue;
@@ -9261,8 +8726,11 @@ Call_expression::result_count() const
 Temporary_statement*
 Call_expression::result(size_t i) const
 {
-  go_assert(this->results_ != NULL
-           && this->results_->size() > i);
+  if (this->results_ == NULL || this->results_->size() <= i)
+    {
+      go_assert(saw_errors());
+      return NULL;
+    }
   return (*this->results_)[i];
 }
 
@@ -9410,7 +8878,7 @@ Call_expression::determining_types()
 bool
 Call_expression::check_argument_type(int i, const Type* parameter_type,
                                     const Type* argument_type,
-                                    source_location argument_location,
+                                    Location argument_location,
                                     bool issued_error)
 {
   std::string reason;
@@ -9442,6 +8910,9 @@ Call_expression::check_argument_type(int i, const Type* parameter_type,
 void
 Call_expression::do_check_types(Gogo*)
 {
+  if (this->classification() == EXPRESSION_ERROR)
+    return;
+
   Function_type* fntype = this->get_function_type();
   if (fntype == NULL)
     {
@@ -9477,7 +8948,17 @@ Call_expression::do_check_types(Gogo*)
     }
 
   // Note that varargs was handled by the lower_varargs() method, so
-  // we don't have to worry about it here.
+  // we don't have to worry about it here unless something is wrong.
+  if (this->is_varargs_ && !this->varargs_are_lowered_)
+    {
+      if (!fntype->is_varargs())
+       {
+         error_at(this->location(),
+                  _("invalid use of %<...%> calling non-variadic function"));
+         this->set_is_error();
+         return;
+       }
+    }
 
   const Typed_identifier_list* parameters = fntype->parameters();
   if (this->args_ == NULL)
@@ -9559,7 +9040,7 @@ Call_expression::do_get_tree(Translate_context* context)
     return error_mark_node;
 
   Gogo* gogo = context->gogo();
-  source_location location = this->location();
+  Location location = this->location();
 
   Func_expression* func = this->fn_->func_expression();
   Interface_field_reference_expression* interface_method =
@@ -9652,12 +9133,13 @@ Call_expression::do_get_tree(Translate_context* context)
       tree fnt = type_to_tree(fntype->get_backend(gogo));
       if (fnt == error_mark_node)
        return error_mark_node;
-      fn = fold_convert_loc(location, fnt, fn);
+      fn = fold_convert_loc(location.gcc_location(), fnt, fn);
     }
 
   // This is to support builtin math functions when using 80387 math.
   tree excess_type = NULL_TREE;
-  if (TREE_CODE(fndecl) == FUNCTION_DECL
+  if (optimize
+      && TREE_CODE(fndecl) == FUNCTION_DECL
       && DECL_IS_BUILTIN(fndecl)
       && DECL_BUILT_IN_CLASS(fndecl) == BUILT_IN_NORMAL
       && nargs > 0
@@ -9675,9 +9157,14 @@ Call_expression::do_get_tree(Translate_context* context)
            excess_type = NULL_TREE;
          else
            {
-             fn = build_fold_addr_expr_loc(location, excess_fndecl);
+             fn = build_fold_addr_expr_loc(location.gcc_location(),
+                                            excess_fndecl);
              for (int i = 0; i < nargs; ++i)
-               args[i] = ::convert(excess_type, args[i]);
+               {
+                 if (SCALAR_FLOAT_TYPE_P(TREE_TYPE(args[i]))
+                     || COMPLEX_FLOAT_TYPE_P(TREE_TYPE(args[i])))
+                   args[i] = ::convert(excess_type, args[i]);
+               }
            }
        }
     }
@@ -9686,7 +9173,7 @@ Call_expression::do_get_tree(Translate_context* context)
                              fn, nargs, args);
   delete[] args;
 
-  SET_EXPR_LOCATION(ret, location);
+  SET_EXPR_LOCATION(ret, location.gcc_location());
 
   if (has_closure)
     {
@@ -9702,7 +9189,7 @@ Call_expression::do_get_tree(Translate_context* context)
   if (TREE_TYPE(ret) == ptr_type_node)
     {
       tree t = type_to_tree(this->type()->base()->get_backend(gogo));
-      ret = fold_convert_loc(location, t, ret);
+      ret = fold_convert_loc(location.gcc_location(), t, ret);
     }
 
   if (excess_type != NULL_TREE)
@@ -9735,7 +9222,7 @@ Call_expression::set_results(Translate_context* context, tree call_tree)
       return call_tree;
     }
 
-  source_location loc = this->location();
+  Location loc = this->location();
   tree field = TYPE_FIELDS(TREE_TYPE(call_tree));
   size_t rc = this->result_count();
   for (size_t i = 0; i < rc; ++i, field = DECL_CHAIN(field))
@@ -9743,17 +9230,22 @@ Call_expression::set_results(Translate_context* context, tree call_tree)
       go_assert(field != NULL_TREE);
 
       Temporary_statement* temp = this->result(i);
+      if (temp == NULL)
+       {
+         go_assert(saw_errors());
+         return error_mark_node;
+       }
       Temporary_reference_expression* ref =
        Expression::make_temporary_reference(temp, loc);
       ref->set_is_lvalue();
       tree temp_tree = ref->get_tree(context);
       if (temp_tree == error_mark_node)
-       continue;
+       return error_mark_node;
 
-      tree val_tree = build3_loc(loc, COMPONENT_REF, TREE_TYPE(field),
-                                call_tree, field, NULL_TREE);
-      tree set_tree = build2_loc(loc, MODIFY_EXPR, void_type_node, temp_tree,
-                                val_tree);
+      tree val_tree = build3_loc(loc.gcc_location(), COMPONENT_REF,
+                                 TREE_TYPE(field), call_tree, field, NULL_TREE);
+      tree set_tree = build2_loc(loc.gcc_location(), MODIFY_EXPR,
+                                 void_type_node, temp_tree, val_tree);
 
       append_to_statement_list(set_tree, &stmt_list);
     }
@@ -9779,7 +9271,7 @@ Call_expression::do_dump_expression(Ast_dump_context* ast_dump_context) const
 
 Call_expression*
 Expression::make_call(Expression* fn, Expression_list* args, bool is_varargs,
-                     source_location location)
+                     Location location)
 {
   return new Call_expression(fn, args, is_varargs, location);
 }
@@ -9922,8 +9414,17 @@ tree
 Call_result_expression::do_get_tree(Translate_context* context)
 {
   Call_expression* ce = this->call_->call_expression();
-  go_assert(ce != NULL);
+  if (ce == NULL)
+    {
+      go_assert(this->call_->is_error_expression());
+      return error_mark_node;
+    }
   Temporary_statement* ts = ce->result(this->index_);
+  if (ts == NULL)
+    {
+      go_assert(saw_errors());
+      return error_mark_node;
+    }
   Expression* ref = Expression::make_temporary_reference(ts, this->location());
   return ref->get_tree(context);
 }
@@ -9971,7 +9472,7 @@ Index_expression::do_traverse(Traverse* traverse)
 Expression*
 Index_expression::do_lower(Gogo*, Named_object*, Statement_inserter*, int)
 {
-  source_location location = this->location();
+  Location location = this->location();
   Expression* left = this->left_;
   Expression* start = this->start_;
   Expression* end = this->end_;
@@ -10051,7 +9552,7 @@ Index_expression::do_dump_expression(Ast_dump_context* ast_dump_context)
 
 Expression*
 Expression::make_index(Expression* left, Expression* start, Expression* end,
-                      source_location location)
+                      Location location)
 {
   return new Index_expression(left, start, end, location);
 }
@@ -10062,7 +9563,7 @@ class Array_index_expression : public Expression
 {
  public:
   Array_index_expression(Expression* array, Expression* start,
-                        Expression* end, source_location location)
+                        Expression* end, Location location)
     : Expression(EXPRESSION_ARRAY_INDEX, location),
       array_(array), start_(start), end_(end), type_(NULL)
   { }
@@ -10202,16 +9703,14 @@ Array_index_expression::do_check_types(Gogo*)
   unsigned int int_bits =
     Type::lookup_integer_type("int")->integer_type()->bits();
 
-  Type* dummy;
+  Numeric_constant lvalnc;
   mpz_t lval;
-  mpz_init(lval);
   bool lval_valid = (array_type->length() != NULL
-                    && array_type->length()->integer_constant_value(true,
-                                                                    lval,
-                                                                    &dummy));
+                    && array_type->length()->numeric_constant_value(&lvalnc)
+                    && lvalnc.to_int(&lval));
+  Numeric_constant inc;
   mpz_t ival;
-  mpz_init(ival);
-  if (this->start_->integer_constant_value(true, ival, &dummy))
+  if (this->start_->numeric_constant_value(&inc) && inc.to_int(&ival))
     {
       if (mpz_sgn(ival) < 0
          || mpz_sizeinbase(ival, 2) >= int_bits
@@ -10223,29 +9722,33 @@ Array_index_expression::do_check_types(Gogo*)
          error_at(this->start_->location(), "array index out of bounds");
          this->set_is_error();
        }
+      mpz_clear(ival);
     }
   if (this->end_ != NULL && !this->end_->is_nil_expression())
     {
-      if (this->end_->integer_constant_value(true, ival, &dummy))
+      Numeric_constant enc;
+      mpz_t eval;
+      if (this->end_->numeric_constant_value(&enc) && enc.to_int(&eval))
        {
-         if (mpz_sgn(ival) < 0
-             || mpz_sizeinbase(ival, 2) >= int_bits
-             || (lval_valid && mpz_cmp(ival, lval) > 0))
+         if (mpz_sgn(eval) < 0
+             || mpz_sizeinbase(eval, 2) >= int_bits
+             || (lval_valid && mpz_cmp(eval, lval) > 0))
            {
              error_at(this->end_->location(), "array index out of bounds");
              this->set_is_error();
            }
+         mpz_clear(eval);
        }
     }
-  mpz_clear(ival);
-  mpz_clear(lval);
+  if (lval_valid)
+    mpz_clear(lval);
 
   // A slice of an array requires an addressable array.  A slice of a
   // slice is always possible.
   if (this->end_ != NULL && !array_type->is_slice_type())
     {
       if (!this->array_->is_addressable())
-       this->report_error(_("array is not addressable"));
+       this->report_error(_("slice of unaddressable value"));
       else
        this->array_->address_taken(true);
     }
@@ -10275,7 +9778,7 @@ tree
 Array_index_expression::do_get_tree(Translate_context* context)
 {
   Gogo* gogo = context->gogo();
-  source_location loc = this->location();
+  Location loc = this->location();
 
   Array_type* array_type = this->array_->type()->array_type();
   if (array_type == NULL)
@@ -10294,11 +9797,28 @@ Array_index_expression::do_get_tree(Translate_context* context)
 
   if (array_type->length() == NULL && !DECL_P(array_tree))
     array_tree = save_expr(array_tree);
-  tree length_tree = array_type->length_tree(gogo, array_tree);
-  if (length_tree == error_mark_node)
-    return error_mark_node;
-  length_tree = save_expr(length_tree);
-  tree length_type = TREE_TYPE(length_tree);
+
+  tree length_tree = NULL_TREE;
+  if (this->end_ == NULL || this->end_->is_nil_expression())
+    {
+      length_tree = array_type->length_tree(gogo, array_tree);
+      if (length_tree == error_mark_node)
+       return error_mark_node;
+      length_tree = save_expr(length_tree);
+    }
+
+  tree capacity_tree = NULL_TREE;
+  if (this->end_ != NULL)
+    {
+      capacity_tree = array_type->capacity_tree(gogo, array_tree);
+      if (capacity_tree == error_mark_node)
+       return error_mark_node;
+      capacity_tree = save_expr(capacity_tree);
+    }
+
+  tree length_type = (length_tree != NULL_TREE
+                     ? TREE_TYPE(length_tree)
+                     : TREE_TYPE(capacity_tree));
 
   tree bad_index = boolean_false_node;
 
@@ -10313,14 +9833,17 @@ Array_index_expression::do_get_tree(Translate_context* context)
   bad_index = Expression::check_bounds(start_tree, length_type, bad_index,
                                       loc);
 
-  start_tree = fold_convert_loc(loc, length_type, start_tree);
-  bad_index = fold_build2_loc(loc, TRUTH_OR_EXPR, boolean_type_node, bad_index,
-                             fold_build2_loc(loc,
+  start_tree = fold_convert_loc(loc.gcc_location(), length_type, start_tree);
+  bad_index = fold_build2_loc(loc.gcc_location(), TRUTH_OR_EXPR,
+                              boolean_type_node, bad_index,
+                             fold_build2_loc(loc.gcc_location(),
                                              (this->end_ == NULL
                                               ? GE_EXPR
                                               : GT_EXPR),
                                              boolean_type_node, start_tree,
-                                             length_tree));
+                                             (this->end_ == NULL
+                                              ? length_tree
+                                              : capacity_tree)));
 
   int code = (array_type->length() != NULL
              ? (this->end_ == NULL
@@ -10339,7 +9862,7 @@ Array_index_expression::do_get_tree(Translate_context* context)
                          build3(COND_EXPR, void_type_node,
                                 bad_index, crash, NULL_TREE),
                          start_tree);
-      start_tree = fold_convert_loc(loc, sizetype, start_tree);
+      start_tree = fold_convert_loc(loc.gcc_location(), sizetype, start_tree);
 
       if (array_type->length() != NULL)
        {
@@ -10357,9 +9880,9 @@ Array_index_expression::do_get_tree(Translate_context* context)
          if (element_type_tree == error_mark_node)
            return error_mark_node;
          tree element_size = TYPE_SIZE_UNIT(element_type_tree);
-         tree offset = fold_build2_loc(loc, MULT_EXPR, sizetype,
+         tree offset = fold_build2_loc(loc.gcc_location(), MULT_EXPR, sizetype,
                                        start_tree, element_size);
-         tree ptr = fold_build2_loc(loc, POINTER_PLUS_EXPR,
+         tree ptr = fold_build2_loc(loc.gcc_location(), POINTER_PLUS_EXPR,
                                     TREE_TYPE(values), values, offset);
          return build_fold_indirect_ref(ptr);
        }
@@ -10367,11 +9890,6 @@ Array_index_expression::do_get_tree(Translate_context* context)
 
   // Array slice.
 
-  tree capacity_tree = array_type->capacity_tree(gogo, array_tree);
-  if (capacity_tree == error_mark_node)
-    return error_mark_node;
-  capacity_tree = fold_convert_loc(loc, length_type, capacity_tree);
-
   tree end_tree;
   if (this->end_->is_nil_expression())
     end_tree = length_tree;
@@ -10388,18 +9906,18 @@ Array_index_expression::do_get_tree(Translate_context* context)
       bad_index = Expression::check_bounds(end_tree, length_type, bad_index,
                                           loc);
 
-      end_tree = fold_convert_loc(loc, length_type, end_tree);
+      end_tree = fold_convert_loc(loc.gcc_location(), length_type, end_tree);
 
-      capacity_tree = save_expr(capacity_tree);
-      tree bad_end = fold_build2_loc(loc, TRUTH_OR_EXPR, boolean_type_node,
-                                    fold_build2_loc(loc, LT_EXPR,
-                                                    boolean_type_node,
+      tree bad_end = fold_build2_loc(loc.gcc_location(), TRUTH_OR_EXPR,
+                                     boolean_type_node,
+                                    fold_build2_loc(loc.gcc_location(),
+                                                     LT_EXPR, boolean_type_node,
                                                     end_tree, start_tree),
-                                    fold_build2_loc(loc, GT_EXPR,
-                                                    boolean_type_node,
+                                    fold_build2_loc(loc.gcc_location(),
+                                                     GT_EXPR, boolean_type_node,
                                                     end_tree, capacity_tree));
-      bad_index = fold_build2_loc(loc, TRUTH_OR_EXPR, boolean_type_node,
-                                 bad_index, bad_end);
+      bad_index = fold_build2_loc(loc.gcc_location(), TRUTH_OR_EXPR,
+                                  boolean_type_node, bad_index, bad_end);
     }
 
   Type* element_type = array_type->element_type();
@@ -10408,23 +9926,25 @@ Array_index_expression::do_get_tree(Translate_context* context)
     return error_mark_node;
   tree element_size = TYPE_SIZE_UNIT(element_type_tree);
 
-  tree offset = fold_build2_loc(loc, MULT_EXPR, sizetype,
-                               fold_convert_loc(loc, sizetype, start_tree),
+  tree offset = fold_build2_loc(loc.gcc_location(), MULT_EXPR, sizetype,
+                               fold_convert_loc(loc.gcc_location(), sizetype,
+                                                 start_tree),
                                element_size);
 
   tree value_pointer = array_type->value_pointer_tree(gogo, array_tree);
   if (value_pointer == error_mark_node)
     return error_mark_node;
 
-  value_pointer = fold_build2_loc(loc, POINTER_PLUS_EXPR,
+  value_pointer = fold_build2_loc(loc.gcc_location(), POINTER_PLUS_EXPR,
                                  TREE_TYPE(value_pointer),
                                  value_pointer, offset);
 
-  tree result_length_tree = fold_build2_loc(loc, MINUS_EXPR, length_type,
-                                           end_tree, start_tree);
+  tree result_length_tree = fold_build2_loc(loc.gcc_location(), MINUS_EXPR,
+                                            length_type, end_tree, start_tree);
 
-  tree result_capacity_tree = fold_build2_loc(loc, MINUS_EXPR, length_type,
-                                             capacity_tree, start_tree);
+  tree result_capacity_tree = fold_build2_loc(loc.gcc_location(), MINUS_EXPR,
+                                              length_type, capacity_tree,
+                                              start_tree);
 
   tree struct_tree = type_to_tree(this->type()->get_backend(gogo));
   go_assert(TREE_CODE(struct_tree) == RECORD_TYPE);
@@ -10441,13 +9961,15 @@ Array_index_expression::do_get_tree(Translate_context* context)
   field = DECL_CHAIN(field);
   go_assert(strcmp(IDENTIFIER_POINTER(DECL_NAME(field)), "__count") == 0);
   elt->index = field;
-  elt->value = fold_convert_loc(loc, TREE_TYPE(field), result_length_tree);
+  elt->value = fold_convert_loc(loc.gcc_location(), TREE_TYPE(field),
+                                result_length_tree);
 
   elt = VEC_quick_push(constructor_elt, init, NULL);
   field = DECL_CHAIN(field);
   go_assert(strcmp(IDENTIFIER_POINTER(DECL_NAME(field)), "__capacity") == 0);
   elt->index = field;
-  elt->value = fold_convert_loc(loc, TREE_TYPE(field), result_capacity_tree);
+  elt->value = fold_convert_loc(loc.gcc_location(), TREE_TYPE(field),
+                                result_capacity_tree);
 
   tree constructor = build_constructor(struct_tree, init);
 
@@ -10456,7 +9978,8 @@ Array_index_expression::do_get_tree(Translate_context* context)
       && TREE_CONSTANT(result_capacity_tree))
     TREE_CONSTANT(constructor) = 1;
 
-  return fold_build2_loc(loc, COMPOUND_EXPR, TREE_TYPE(constructor),
+  return fold_build2_loc(loc.gcc_location(), COMPOUND_EXPR,
+                         TREE_TYPE(constructor),
                         build3(COND_EXPR, void_type_node,
                                bad_index, crash, NULL_TREE),
                         constructor);
@@ -10476,15 +9999,8 @@ Array_index_expression::do_dump_expression(Ast_dump_context* ast_dump_context)
 
 Expression*
 Expression::make_array_index(Expression* array, Expression* start,
-                            Expression* end, source_location location)
+                            Expression* end, Location location)
 {
-  // Taking a slice of a composite literal requires moving the literal
-  // onto the heap.
-  if (end != NULL && array->is_composite_literal())
-    {
-      array = Expression::make_heap_composite(array, location);
-      array = Expression::make_unary(OPERATOR_MULT, array, location);
-    }
   return new Array_index_expression(array, start, end, location);
 }
 
@@ -10494,7 +10010,7 @@ class String_index_expression : public Expression
 {
  public:
   String_index_expression(Expression* string, Expression* start,
-                         Expression* end, source_location location)
+                         Expression* end, Location location)
     : Expression(EXPRESSION_STRING_INDEX, location),
       string_(string), start_(start), end_(end)
   { }
@@ -10600,10 +10116,9 @@ String_index_expression::do_check_types(Gogo*)
   std::string sval;
   bool sval_valid = this->string_->string_constant_value(&sval);
 
+  Numeric_constant inc;
   mpz_t ival;
-  mpz_init(ival);
-  Type* dummy;
-  if (this->start_->integer_constant_value(true, ival, &dummy))
+  if (this->start_->numeric_constant_value(&inc) && inc.to_int(&ival))
     {
       if (mpz_sgn(ival) < 0
          || (sval_valid && mpz_cmp_ui(ival, sval.length()) >= 0))
@@ -10611,20 +10126,23 @@ String_index_expression::do_check_types(Gogo*)
          error_at(this->start_->location(), "string index out of bounds");
          this->set_is_error();
        }
+      mpz_clear(ival);
     }
   if (this->end_ != NULL && !this->end_->is_nil_expression())
     {
-      if (this->end_->integer_constant_value(true, ival, &dummy))
+      Numeric_constant enc;
+      mpz_t eval;
+      if (this->end_->numeric_constant_value(&enc) && enc.to_int(&eval))
        {
-         if (mpz_sgn(ival) < 0
-             || (sval_valid && mpz_cmp_ui(ival, sval.length()) > 0))
+         if (mpz_sgn(eval) < 0
+             || (sval_valid && mpz_cmp_ui(eval, sval.length()) > 0))
            {
              error_at(this->end_->location(), "string index out of bounds");
              this->set_is_error();
            }
+         mpz_clear(eval);
        }
     }
-  mpz_clear(ival);
 }
 
 // Get a tree for a string index.
@@ -10632,7 +10150,7 @@ String_index_expression::do_check_types(Gogo*)
 tree
 String_index_expression::do_get_tree(Translate_context* context)
 {
-  source_location loc = this->location();
+  Location loc = this->location();
 
   tree string_tree = this->string_->get_tree(context);
   if (string_tree == error_mark_node)
@@ -10661,7 +10179,7 @@ String_index_expression::do_get_tree(Translate_context* context)
   bad_index = Expression::check_bounds(start_tree, length_type, bad_index,
                                       loc);
 
-  start_tree = fold_convert_loc(loc, length_type, start_tree);
+  start_tree = fold_convert_loc(loc.gcc_location(), length_type, start_tree);
 
   int code = (this->end_ == NULL
              ? RUNTIME_ERROR_STRING_INDEX_OUT_OF_BOUNDS
@@ -10670,17 +10188,19 @@ String_index_expression::do_get_tree(Translate_context* context)
 
   if (this->end_ == NULL)
     {
-      bad_index = fold_build2_loc(loc, TRUTH_OR_EXPR, boolean_type_node,
-                                 bad_index,
-                                 fold_build2_loc(loc, GE_EXPR,
+      bad_index = fold_build2_loc(loc.gcc_location(), TRUTH_OR_EXPR,
+                                  boolean_type_node, bad_index,
+                                 fold_build2_loc(loc.gcc_location(), GE_EXPR,
                                                  boolean_type_node,
                                                  start_tree, length_tree));
 
       tree bytes_tree = String_type::bytes_tree(context->gogo(), string_tree);
-      tree ptr = fold_build2_loc(loc, POINTER_PLUS_EXPR, TREE_TYPE(bytes_tree),
+      tree ptr = fold_build2_loc(loc.gcc_location(), POINTER_PLUS_EXPR,
+                                 TREE_TYPE(bytes_tree),
                                 bytes_tree,
-                                fold_convert_loc(loc, sizetype, start_tree));
-      tree index = build_fold_indirect_ref_loc(loc, ptr);
+                                fold_convert_loc(loc.gcc_location(), sizetype,
+                                                  start_tree));
+      tree index = build_fold_indirect_ref_loc(loc.gcc_location(), ptr);
 
       return build2(COMPOUND_EXPR, TREE_TYPE(index),
                    build3(COND_EXPR, void_type_node,
@@ -10705,7 +10225,8 @@ String_index_expression::do_get_tree(Translate_context* context)
          bad_index = Expression::check_bounds(end_tree, length_type,
                                               bad_index, loc);
 
-         end_tree = fold_convert_loc(loc, length_type, end_tree);
+         end_tree = fold_convert_loc(loc.gcc_location(), length_type,
+                                      end_tree);
        }
 
       static tree strslice_fndecl;
@@ -10750,7 +10271,7 @@ String_index_expression::do_dump_expression(Ast_dump_context* ast_dump_context)
 
 Expression*
 Expression::make_string_index(Expression* string, Expression* start,
-                             Expression* end, source_location location)
+                             Expression* end, Location location)
 {
   return new String_index_expression(string, start, end, location);
 }
@@ -10906,14 +10427,16 @@ Map_index_expression::get_value_pointer(Translate_context* context,
     }
   else
     {
-      tmp = build_decl(this->location(), VAR_DECL, create_tmp_var_name("M"),
+      tmp = build_decl(this->location().gcc_location(), VAR_DECL,
+                       create_tmp_var_name("M"),
                       TREE_TYPE(index_tree));
       DECL_EXTERNAL(tmp) = 0;
       TREE_PUBLIC(tmp) = 0;
       TREE_STATIC(tmp) = 1;
       DECL_ARTIFICIAL(tmp) = 1;
       if (!TREE_CONSTANT(index_tree))
-       make_tmp = fold_build2_loc(this->location(), INIT_EXPR, void_type_node,
+       make_tmp = fold_build2_loc(this->location().gcc_location(),
+                                   INIT_EXPR, void_type_node,
                                   tmp, index_tree);
       else
        {
@@ -10924,9 +10447,10 @@ Map_index_expression::get_value_pointer(Translate_context* context,
        }
       rest_of_decl_compilation(tmp, 1, 0);
     }
-  tree tmpref = fold_convert_loc(this->location(), const_ptr_type_node,
-                                build_fold_addr_expr_loc(this->location(),
-                                                         tmp));
+  tree tmpref =
+    fold_convert_loc(this->location().gcc_location(), const_ptr_type_node,
+                     build_fold_addr_expr_loc(this->location().gcc_location(),
+                                              tmp));
 
   static tree map_index_fndecl;
   tree call = Gogo::call_builtin(&map_index_fndecl,
@@ -10954,7 +10478,8 @@ Map_index_expression::get_value_pointer(Translate_context* context,
     return error_mark_node;
   tree ptr_val_type_tree = build_pointer_type(val_type_tree);
 
-  tree ret = fold_convert_loc(this->location(), ptr_val_type_tree, call);
+  tree ret = fold_convert_loc(this->location().gcc_location(),
+                              ptr_val_type_tree, call);
   if (make_tmp != NULL_TREE)
     ret = build2(COMPOUND_EXPR, ptr_val_type_tree, make_tmp, ret);
   return ret;
@@ -10974,7 +10499,7 @@ Map_index_expression::do_dump_expression(Ast_dump_context* ast_dump_context)
 
 Map_index_expression*
 Expression::make_map_index(Expression* map, Expression* index,
-                          source_location location)
+                          Location location)
 {
   return new Map_index_expression(map, index, location);
 }
@@ -11050,7 +10575,7 @@ Field_reference_expression::do_dump_expression(
 
 Field_reference_expression*
 Expression::make_field_reference(Expression* expr, unsigned int field_index,
-                                source_location location)
+                                Location location)
 {
   return new Field_reference_expression(expr, field_index, location);
 }
@@ -11189,7 +10714,8 @@ Interface_field_reference_expression::do_check_types(Gogo*)
 tree
 Interface_field_reference_expression::do_get_tree(Translate_context*)
 {
-  go_unreachable();
+  error_at(this->location(), "reference to method other than calling it");
+  return error_mark_node;
 }
 
 // Dump ast representation for an interface field reference.
@@ -11207,7 +10733,7 @@ Interface_field_reference_expression::do_dump_expression(
 Expression*
 Expression::make_interface_field_reference(Expression* expr,
                                           const std::string& field,
-                                          source_location location)
+                                          Location location)
 {
   return new Interface_field_reference_expression(expr, field, location);
 }
@@ -11219,7 +10745,7 @@ class Selector_expression : public Parser_expression
 {
  public:
   Selector_expression(Expression* left, const std::string& name,
-                     source_location location)
+                     Location location)
     : Parser_expression(EXPRESSION_SELECTOR, location),
       left_(left), name_(name)
   { }
@@ -11272,7 +10798,7 @@ Selector_expression::do_lower(Gogo* gogo, Named_object*, Statement_inserter*,
 Expression*
 Selector_expression::lower_method_expression(Gogo* gogo)
 {
-  source_location location = this->location();
+  Location location = this->location();
   Type* type = this->left_->type();
   const std::string& name(this->name_);
 
@@ -11348,10 +10874,21 @@ Selector_expression::lower_method_expression(Gogo* gogo)
   const Typed_identifier_list* method_parameters = method_type->parameters();
   if (method_parameters != NULL)
     {
+      int i = 0;
       for (Typed_identifier_list::const_iterator p = method_parameters->begin();
           p != method_parameters->end();
-          ++p)
-       parameters->push_back(*p);
+          ++p, ++i)
+       {
+         if (!p->name().empty())
+           parameters->push_back(*p);
+         else
+           {
+             char buf[20];
+             snprintf(buf, sizeof buf, "$param%d", i);
+             parameters->push_back(Typed_identifier(buf, p->type(),
+                                                    p->location()));
+           }
+       }
     }
 
   const Typed_identifier_list* method_results = method_type->results();
@@ -11411,14 +10948,14 @@ Selector_expression::lower_method_expression(Gogo* gogo)
     }
 
   Expression_list* args;
-  if (method_parameters == NULL)
+  if (parameters->size() <= 1)
     args = NULL;
   else
     {
       args = new Expression_list();
-      for (Typed_identifier_list::const_iterator p = method_parameters->begin();
-          p != method_parameters->end();
-          ++p)
+      Typed_identifier_list::const_iterator p = parameters->begin();
+      ++p;
+      for (; p != parameters->end(); ++p)
        {
          vno = gogo->lookup(p->name(), NULL);
          go_assert(vno != NULL);
@@ -11477,7 +11014,7 @@ Selector_expression::do_dump_expression(Ast_dump_context* ast_dump_context)
 
 Expression*
 Expression::make_selector(Expression* left, const std::string& name,
-                         source_location location)
+                         Location location)
 {
   return new Selector_expression(left, name, location);
 }
@@ -11487,7 +11024,7 @@ Expression::make_selector(Expression* left, const std::string& name,
 class Allocation_expression : public Expression
 {
  public:
-  Allocation_expression(Type* type, source_location location)
+  Allocation_expression(Type* type, Location location)
     : Expression(EXPRESSION_ALLOCATION, location),
       type_(type)
   { }
@@ -11550,7 +11087,7 @@ Allocation_expression::do_dump_expression(Ast_dump_context* ast_dump_context)
 // Make an allocation expression.
 
 Expression*
-Expression::make_allocation(Type* type, source_location location)
+Expression::make_allocation(Type* type, Location location)
 {
   return new Allocation_expression(type, location);
 }
@@ -11561,11 +11098,17 @@ class Struct_construction_expression : public Expression
 {
  public:
   Struct_construction_expression(Type* type, Expression_list* vals,
-                                source_location location)
+                                Location location)
     : Expression(EXPRESSION_STRUCT_CONSTRUCTION, location),
-      type_(type), vals_(vals)
+      type_(type), vals_(vals), traverse_order_(NULL)
   { }
 
+  // Set the traversal order, used to ensure that we implement the
+  // order of evaluation rules.  Takes ownership of the argument.
+  void
+  set_traverse_order(std::vector<int>* traverse_order)
+  { this->traverse_order_ = traverse_order; }
+
   // Return whether this is a constant initializer.
   bool
   is_constant_struct() const;
@@ -11587,14 +11130,14 @@ class Struct_construction_expression : public Expression
   Expression*
   do_copy()
   {
-    return new Struct_construction_expression(this->type_, this->vals_->copy(),
-                                             this->location());
+    Struct_construction_expression* ret =
+      new Struct_construction_expression(this->type_, this->vals_->copy(),
+                                        this->location());
+    if (this->traverse_order_ != NULL)
+      ret->set_traverse_order(this->traverse_order_);
+    return ret;
   }
 
-  bool
-  do_is_addressable() const
-  { return true; }
-
   tree
   do_get_tree(Translate_context*);
 
@@ -11610,6 +11153,9 @@ class Struct_construction_expression : public Expression
   // The list of values, in order of the fields in the struct.  A NULL
   // entry means that the field should be zero-initialized.
   Expression_list* vals_;
+  // If not NULL, the order in which to traverse vals_.  This is used
+  // so that we implement the order of evaluation rules correctly.
+  std::vector<int>* traverse_order_;
 };
 
 // Traversal.
@@ -11617,12 +11163,29 @@ class Struct_construction_expression : public Expression
 int
 Struct_construction_expression::do_traverse(Traverse* traverse)
 {
-  if (this->vals_ != NULL
-      && this->vals_->traverse(traverse) == TRAVERSE_EXIT)
-    return TRAVERSE_EXIT;
-  if (Type::traverse(this->type_, traverse) == TRAVERSE_EXIT)
-    return TRAVERSE_EXIT;
-  return TRAVERSE_CONTINUE;
+  if (this->vals_ != NULL)
+    {
+      if (this->traverse_order_ == NULL)
+       {
+         if (this->vals_->traverse(traverse) == TRAVERSE_EXIT)
+           return TRAVERSE_EXIT;
+       }
+      else
+       {
+         for (std::vector<int>::const_iterator p =
+                this->traverse_order_->begin();
+              p != this->traverse_order_->end();
+              ++p)
+           {
+             if (Expression::traverse(&this->vals_->at(*p), traverse)
+                 == TRAVERSE_EXIT)
+               return TRAVERSE_EXIT;
+           }
+       }
+    }
+  if (Type::traverse(this->type_, traverse) == TRAVERSE_EXIT)
+    return TRAVERSE_EXIT;
+  return TRAVERSE_CONTINUE;
 }
 
 // Return whether this is a constant initializer.
@@ -11832,7 +11395,7 @@ Struct_construction_expression::do_dump_expression(
 
 Expression*
 Expression::make_struct_composite_literal(Type* type, Expression_list* vals,
-                                         source_location location)
+                                         Location location)
 {
   go_assert(type->struct_type() != NULL);
   return new Struct_construction_expression(type, vals, location);
@@ -11846,11 +11409,12 @@ class Array_construction_expression : public Expression
 {
  protected:
   Array_construction_expression(Expression_classification classification,
-                               Type* type, Expression_list* vals,
-                               source_location location)
+                               Type* type,
+                               const std::vector<unsigned long>* indexes,
+                               Expression_list* vals, Location location)
     : Expression(classification, location),
-      type_(type), vals_(vals)
-  { }
+      type_(type), indexes_(indexes), vals_(vals)
+  { go_assert(indexes == NULL || indexes->size() == vals->size()); }
 
  public:
   // Return whether this is a constant initializer.
@@ -11876,13 +11440,14 @@ protected:
   void
   do_check_types(Gogo*);
 
-  bool
-  do_is_addressable() const
-  { return true; }
-
   void
   do_export(Export*) const;
 
+  // The indexes.
+  const std::vector<unsigned long>*
+  indexes()
+  { return this->indexes_; }
+
   // The list of values.
   Expression_list*
   vals()
@@ -11898,7 +11463,10 @@ protected:
  private:
   // The type of the array to construct.
   Type* type_;
-  // The list of values.
+  // The list of indexes into the array, one for each value.  This may
+  // be NULL, in which case the indexes start at zero and increment.
+  const std::vector<unsigned long>* indexes_;
+  // The list of values.  This may be NULL if there are no values.
   Expression_list* vals_;
 };
 
@@ -11981,20 +11549,6 @@ Array_construction_expression::do_check_types(Gogo*)
          this->set_is_error();
        }
     }
-
-  Expression* length = at->length();
-  if (length != NULL && !length->is_error_expression())
-    {
-      mpz_t val;
-      mpz_init(val);
-      Type* type;
-      if (at->length()->integer_constant_value(true, val, &type))
-       {
-         if (this->vals_->size() > mpz_get_ui(val))
-           this->report_error(_("too many elements in composite literal"));
-       }
-      mpz_clear(val);
-    }
 }
 
 // Get a constructor tree for the array values.
@@ -12012,12 +11566,22 @@ Array_construction_expression::get_constructor_tree(Translate_context* context,
   if (this->vals_ != NULL)
     {
       size_t i = 0;
+      std::vector<unsigned long>::const_iterator pi;
+      if (this->indexes_ != NULL)
+       pi = this->indexes_->begin();
       for (Expression_list::const_iterator pv = this->vals_->begin();
           pv != this->vals_->end();
           ++pv, ++i)
        {
+         if (this->indexes_ != NULL)
+           go_assert(pi != this->indexes_->end());
          constructor_elt* elt = VEC_quick_push(constructor_elt, values, NULL);
-         elt->index = size_int(i);
+
+         if (this->indexes_ == NULL)
+           elt->index = size_int(i);
+         else
+           elt->index = size_int(*pi);
+
          if (*pv == NULL)
            {
              Gogo* gogo = context->gogo();
@@ -12038,7 +11602,11 @@ Array_construction_expression::get_constructor_tree(Translate_context* context,
            return error_mark_node;
          if (!TREE_CONSTANT(elt->value))
            is_constant = false;
+         if (this->indexes_ != NULL)
+           ++pi;
        }
+      if (this->indexes_ != NULL)
+       go_assert(pi == this->indexes_->end());
     }
 
   tree ret = build_constructor(type_tree, values);
@@ -12056,13 +11624,28 @@ Array_construction_expression::do_export(Export* exp) const
   exp->write_type(this->type_);
   if (this->vals_ != NULL)
     {
+      std::vector<unsigned long>::const_iterator pi;
+      if (this->indexes_ != NULL)
+       pi = this->indexes_->begin();
       for (Expression_list::const_iterator pv = this->vals_->begin();
           pv != this->vals_->end();
           ++pv)
        {
          exp->write_c_string(", ");
+
+         if (this->indexes_ != NULL)
+           {
+             char buf[100];
+             snprintf(buf, sizeof buf, "%lu", *pi);
+             exp->write_c_string(buf);
+             exp->write_c_string(":");
+           }
+
          if (*pv != NULL)
            (*pv)->export_expression(exp);
+
+         if (this->indexes_ != NULL)
+           ++pi;
        }
     }
   exp->write_c_string(")");
@@ -12074,8 +11657,7 @@ void
 Array_construction_expression::do_dump_expression(
     Ast_dump_context* ast_dump_context) const
 {
-  Expression* length = this->type_->array_type() != NULL ?
-                        this->type_->array_type()->length() : NULL;
+  Expression* length = this->type_->array_type()->length();
 
   ast_dump_context->ostream() << "[" ;
   if (length != NULL)
@@ -12085,7 +11667,22 @@ Array_construction_expression::do_dump_expression(
   ast_dump_context->ostream() << "]" ;
   ast_dump_context->dump_type(this->type_);
   ast_dump_context->ostream() << "{" ;
-  ast_dump_context->dump_expression_list(this->vals_);
+  if (this->indexes_ == NULL)
+    ast_dump_context->dump_expression_list(this->vals_);
+  else
+    {
+      Expression_list::const_iterator pv = this->vals_->begin();
+      for (std::vector<unsigned long>::const_iterator pi =
+            this->indexes_->begin();
+          pi != this->indexes_->end();
+          ++pi, ++pv)
+       {
+         if (pi != this->indexes_->begin())
+           ast_dump_context->ostream() << ", ";
+         ast_dump_context->ostream() << *pi << ':';
+         ast_dump_context->dump_expression(*pv);
+       }
+    }
   ast_dump_context->ostream() << "}" ;
 
 }
@@ -12096,20 +11693,19 @@ class Fixed_array_construction_expression :
   public Array_construction_expression
 {
  public:
-  Fixed_array_construction_expression(Type* type, Expression_list* vals,
-                                     source_location location)
+  Fixed_array_construction_expression(Type* type,
+                                     const std::vector<unsigned long>* indexes,
+                                     Expression_list* vals, Location location)
     : Array_construction_expression(EXPRESSION_FIXED_ARRAY_CONSTRUCTION,
-                                   type, vals, location)
-  {
-    go_assert(type->array_type() != NULL
-              && type->array_type()->length() != NULL);
-  }
+                                   type, indexes, vals, location)
+  { go_assert(type->array_type() != NULL && !type->is_slice_type()); }
 
  protected:
   Expression*
   do_copy()
   {
     return new Fixed_array_construction_expression(this->type(),
+                                                  this->indexes(),
                                                   (this->vals() == NULL
                                                    ? NULL
                                                    : this->vals()->copy()),
@@ -12118,9 +11714,6 @@ class Fixed_array_construction_expression :
 
   tree
   do_get_tree(Translate_context*);
-
-  void
-  do_dump_expression(Ast_dump_context*);
 };
 
 // Return a tree for constructing a fixed array.
@@ -12133,35 +11726,17 @@ Fixed_array_construction_expression::do_get_tree(Translate_context* context)
   return this->get_constructor_tree(context, type_to_tree(btype));
 }
 
-// Dump ast representation of an array construction expressin.
-
-void
-Fixed_array_construction_expression::do_dump_expression(
-    Ast_dump_context* ast_dump_context)
-{
-
-  ast_dump_context->ostream() << "[";
-  ast_dump_context->dump_expression (this->type()->array_type()->length());
-  ast_dump_context->ostream() << "]";
-  ast_dump_context->dump_type(this->type());
-  ast_dump_context->ostream() << "{";
-  ast_dump_context->dump_expression_list(this->vals());
-  ast_dump_context->ostream() << "}";
-
-}
 // Construct an open array.
 
 class Open_array_construction_expression : public Array_construction_expression
 {
  public:
-  Open_array_construction_expression(Type* type, Expression_list* vals,
-                                    source_location location)
+  Open_array_construction_expression(Type* type,
+                                    const std::vector<unsigned long>* indexes,
+                                    Expression_list* vals, Location location)
     : Array_construction_expression(EXPRESSION_OPEN_ARRAY_CONSTRUCTION,
-                                   type, vals, location)
-  {
-    go_assert(type->array_type() != NULL
-              && type->array_type()->length() == NULL);
-  }
+                                   type, indexes, vals, location)
+  { go_assert(type->is_slice_type()); }
 
  protected:
   // Note that taking the address of an open array literal is invalid.
@@ -12170,6 +11745,7 @@ class Open_array_construction_expression : public Array_construction_expression
   do_copy()
   {
     return new Open_array_construction_expression(this->type(),
+                                                 this->indexes(),
                                                  (this->vals() == NULL
                                                   ? NULL
                                                   : this->vals()->copy()),
@@ -12221,13 +11797,18 @@ Open_array_construction_expression::do_get_tree(Translate_context* context)
     }
   else
     {
-      tree max = size_int(this->vals()->size() - 1);
+      unsigned long max_index;
+      if (this->indexes() == NULL)
+       max_index = this->vals()->size() - 1;
+      else
+       max_index = this->indexes()->back();
+      tree max_tree = size_int(max_index);
       tree constructor_type = build_array_type(element_type_tree,
-                                              build_index_type(max));
+                                              build_index_type(max_tree));
       if (constructor_type == error_mark_node)
        return error_mark_node;
       values = this->get_constructor_tree(context, constructor_type);
-      length_tree = size_int(this->vals()->size());
+      length_tree = size_int(max_index + 1);
     }
 
   if (values == error_mark_node)
@@ -12246,7 +11827,7 @@ Open_array_construction_expression::do_get_tree(Translate_context* context)
 
   if (is_constant_initializer)
     {
-      tree tmp = build_decl(this->location(), VAR_DECL,
+      tree tmp = build_decl(this->location().gcc_location(), VAR_DECL,
                            create_tmp_var_name("C"), TREE_TYPE(values));
       DECL_EXTERNAL(tmp) = 0;
       TREE_PUBLIC(tmp) = 0;
@@ -12282,7 +11863,8 @@ Open_array_construction_expression::do_get_tree(Translate_context* context)
       space = save_expr(space);
 
       tree s = fold_convert(build_pointer_type(TREE_TYPE(values)), space);
-      tree ref = build_fold_indirect_ref_loc(this->location(), s);
+      tree ref = build_fold_indirect_ref_loc(this->location().gcc_location(),
+                                             s);
       TREE_THIS_NOTRAP(ref) = 1;
       set = build2(MODIFY_EXPR, void_type_node, ref, values);
     }
@@ -12331,10 +11913,10 @@ Open_array_construction_expression::do_get_tree(Translate_context* context)
 
 Expression*
 Expression::make_slice_composite_literal(Type* type, Expression_list* vals,
-                                        source_location location)
+                                        Location location)
 {
   go_assert(type->is_slice_type());
-  return new Open_array_construction_expression(type, vals, location);
+  return new Open_array_construction_expression(type, NULL, vals, location);
 }
 
 // Construct a map.
@@ -12343,7 +11925,7 @@ class Map_construction_expression : public Expression
 {
  public:
   Map_construction_expression(Type* type, Expression_list* vals,
-                             source_location location)
+                             Location location)
     : Expression(EXPRESSION_MAP_CONSTRUCTION, location),
       type_(type), vals_(vals)
   { go_assert(vals == NULL || vals->size() % 2 == 0); }
@@ -12460,7 +12042,7 @@ tree
 Map_construction_expression::do_get_tree(Translate_context* context)
 {
   Gogo* gogo = context->gogo();
-  source_location loc = this->location();
+  Location loc = this->location();
 
   Map_type* mt = this->type_->map_type();
 
@@ -12472,7 +12054,8 @@ Map_construction_expression::do_get_tree(Translate_context* context)
   tree key_type_tree = type_to_tree(key_type->get_backend(gogo));
   if (key_type_tree == error_mark_node)
     return error_mark_node;
-  tree key_field = build_decl(loc, FIELD_DECL, id, key_type_tree);
+  tree key_field = build_decl(loc.gcc_location(), FIELD_DECL, id,
+                              key_type_tree);
   DECL_CONTEXT(key_field) = struct_type;
   TYPE_FIELDS(struct_type) = key_field;
 
@@ -12481,7 +12064,8 @@ Map_construction_expression::do_get_tree(Translate_context* context)
   tree val_type_tree = type_to_tree(val_type->get_backend(gogo));
   if (val_type_tree == error_mark_node)
     return error_mark_node;
-  tree val_field = build_decl(loc, FIELD_DECL, id, val_type_tree);
+  tree val_field = build_decl(loc.gcc_location(), FIELD_DECL, id,
+                              val_type_tree);
   DECL_CONTEXT(val_field) = struct_type;
   DECL_CHAIN(key_field) = val_field;
 
@@ -12553,19 +12137,21 @@ Map_construction_expression::do_get_tree(Translate_context* context)
        {
          tmp = create_tmp_var(array_type, get_name(array_type));
          DECL_INITIAL(tmp) = init;
-         make_tmp = fold_build1_loc(loc, DECL_EXPR, void_type_node, tmp);
+         make_tmp = fold_build1_loc(loc.gcc_location(), DECL_EXPR,
+                                     void_type_node, tmp);
          TREE_ADDRESSABLE(tmp) = 1;
        }
       else
        {
-         tmp = build_decl(loc, VAR_DECL, create_tmp_var_name("M"), array_type);
+         tmp = build_decl(loc.gcc_location(), VAR_DECL,
+                           create_tmp_var_name("M"), array_type);
          DECL_EXTERNAL(tmp) = 0;
          TREE_PUBLIC(tmp) = 0;
          TREE_STATIC(tmp) = 1;
          DECL_ARTIFICIAL(tmp) = 1;
          if (!TREE_CONSTANT(init))
-           make_tmp = fold_build2_loc(loc, INIT_EXPR, void_type_node, tmp,
-                                      init);
+           make_tmp = fold_build2_loc(loc.gcc_location(), INIT_EXPR,
+                                       void_type_node, tmp, init);
          else
            {
              TREE_READONLY(tmp) = 1;
@@ -12610,7 +12196,8 @@ Map_construction_expression::do_get_tree(Translate_context* context)
   if (make_tmp == NULL)
     ret = call;
   else
-    ret = fold_build2_loc(loc, COMPOUND_EXPR, type_tree, make_tmp, call);
+    ret = fold_build2_loc(loc.gcc_location(), COMPOUND_EXPR, type_tree,
+                          make_tmp, call);
   return ret;
 }
 
@@ -12649,7 +12236,7 @@ class Composite_literal_expression : public Parser_expression
 {
  public:
   Composite_literal_expression(Type* type, int depth, bool has_keys,
-                              Expression_list* vals, source_location location)
+                              Expression_list* vals, Location location)
     : Parser_expression(EXPRESSION_COMPOSITE_LITERAL, location),
       type_(type), depth_(depth), vals_(vals), has_keys_(has_keys)
   { }
@@ -12683,7 +12270,7 @@ class Composite_literal_expression : public Parser_expression
   lower_array(Type*);
 
   Expression*
-  make_array(Type*, Expression_list*);
+  make_array(Type*, const std::vector<unsigned long>*, Expression_list*);
 
   Expression*
   lower_map(Gogo*, Named_object*, Statement_inserter*, Type*);
@@ -12736,14 +12323,23 @@ Composite_literal_expression::do_lower(Gogo* gogo, Named_object* function,
        }
     }
 
+  Type *pt = type->points_to();
+  bool is_pointer = false;
+  if (pt != NULL)
+    {
+      is_pointer = true;
+      type = pt;
+    }
+
+  Expression* ret;
   if (type->is_error())
     return Expression::make_error(this->location());
   else if (type->struct_type() != NULL)
-    return this->lower_struct(gogo, type);
+    ret = this->lower_struct(gogo, type);
   else if (type->array_type() != NULL)
-    return this->lower_array(type);
+    ret = this->lower_array(type);
   else if (type->map_type() != NULL)
-    return this->lower_map(gogo, function, inserter, type);
+    ret = this->lower_map(gogo, function, inserter, type);
   else
     {
       error_at(this->location(),
@@ -12751,6 +12347,11 @@ Composite_literal_expression::do_lower(Gogo* gogo, Named_object* function,
                "for composite literal"));
       return Expression::make_error(this->location());
     }
+
+  if (is_pointer)
+    ret = Expression::make_heap_composite(ret, this->location());
+
+  return ret;
 }
 
 // Lower a struct composite literal.
@@ -12758,13 +12359,33 @@ Composite_literal_expression::do_lower(Gogo* gogo, Named_object* function,
 Expression*
 Composite_literal_expression::lower_struct(Gogo* gogo, Type* type)
 {
-  source_location location = this->location();
+  Location location = this->location();
   Struct_type* st = type->struct_type();
   if (this->vals_ == NULL || !this->has_keys_)
-    return new Struct_construction_expression(type, this->vals_, location);
+    {
+      if (this->vals_ != NULL
+         && !this->vals_->empty()
+         && type->named_type() != NULL
+         && type->named_type()->named_object()->package() != NULL)
+       {
+         for (Struct_field_list::const_iterator pf = st->fields()->begin();
+              pf != st->fields()->end();
+              ++pf)
+           {
+             if (Gogo::is_hidden_name(pf->field_name()))
+               error_at(this->location(),
+                        "assignment of unexported field %qs in %qs literal",
+                        Gogo::message_name(pf->field_name()).c_str(),
+                        type->named_type()->message_name().c_str());
+           }
+       }
+
+      return new Struct_construction_expression(type, this->vals_, location);
+    }
 
   size_t field_count = st->field_count();
   std::vector<Expression*> vals(field_count);
+  std::vector<int>* traverse_order = new(std::vector<int>);
   Expression_list::const_iterator p = this->vals_->begin();
   while (p != this->vals_->end())
     {
@@ -12908,7 +12529,16 @@ Composite_literal_expression::lower_struct(Gogo* gogo, Type* type)
          return Expression::make_error(location);
        }
 
+      if (type->named_type() != NULL
+         && type->named_type()->named_object()->package() != NULL
+         && Gogo::is_hidden_name(sf->field_name()))
+       error_at(name_expr->location(),
+                "assignment of unexported field %qs in %qs literal",
+                Gogo::message_name(sf->field_name()).c_str(),
+                type->named_type()->message_name().c_str());
+
       vals[index] = val;
+      traverse_order->push_back(index);
     }
 
   Expression_list* list = new Expression_list;
@@ -12916,20 +12546,37 @@ Composite_literal_expression::lower_struct(Gogo* gogo, Type* type)
   for (size_t i = 0; i < field_count; ++i)
     list->push_back(vals[i]);
 
-  return new Struct_construction_expression(type, list, location);
+  Struct_construction_expression* ret =
+    new Struct_construction_expression(type, list, location);
+  ret->set_traverse_order(traverse_order);
+  return ret;
 }
 
+// Used to sort an index/value array.
+
+class Index_value_compare
+{
+ public:
+  bool
+  operator()(const std::pair<unsigned long, Expression*>& a,
+            const std::pair<unsigned long, Expression*>& b)
+  { return a.first < b.first; }
+};
+
 // Lower an array composite literal.
 
 Expression*
 Composite_literal_expression::lower_array(Type* type)
 {
-  source_location location = this->location();
+  Location location = this->location();
   if (this->vals_ == NULL || !this->has_keys_)
-    return this->make_array(type, this->vals_);
+    return this->make_array(type, NULL, this->vals_);
 
-  std::vector<Expression*> vals;
-  vals.reserve(this->vals_->size());
+  std::vector<unsigned long>* indexes = new std::vector<unsigned long>;
+  indexes->reserve(this->vals_->size());
+  bool indexes_out_of_order = false;
+  Expression_list* vals = new Expression_list();
+  vals->reserve(this->vals_->size());
   unsigned long index = 0;
   Expression_list::const_iterator p = this->vals_->begin();
   while (p != this->vals_->end())
@@ -12942,104 +12589,141 @@ Composite_literal_expression::lower_array(Type* type)
 
       ++p;
 
-      if (index_expr != NULL)
+      if (index_expr == NULL)
+       {
+         if (!indexes->empty())
+           indexes->push_back(index);
+       }
+      else
        {
-         mpz_t ival;
-         mpz_init(ival);
+         if (indexes->empty() && !vals->empty())
+           {
+             for (size_t i = 0; i < vals->size(); ++i)
+               indexes->push_back(i);
+           }
 
-         Type* dummy;
-         if (!index_expr->integer_constant_value(true, ival, &dummy))
+         Numeric_constant nc;
+         if (!index_expr->numeric_constant_value(&nc))
            {
-             mpz_clear(ival);
              error_at(index_expr->location(),
                       "index expression is not integer constant");
              return Expression::make_error(location);
            }
 
-         if (mpz_sgn(ival) < 0)
+         switch (nc.to_unsigned_long(&index))
            {
-             mpz_clear(ival);
+           case Numeric_constant::NC_UL_VALID:
+             break;
+           case Numeric_constant::NC_UL_NOTINT:
+             error_at(index_expr->location(),
+                      "index expression is not integer constant");
+             return Expression::make_error(location);
+           case Numeric_constant::NC_UL_NEGATIVE:
              error_at(index_expr->location(), "index expression is negative");
              return Expression::make_error(location);
-           }
-
-         index = mpz_get_ui(ival);
-         if (mpz_cmp_ui(ival, index) != 0)
-           {
-             mpz_clear(ival);
+           case Numeric_constant::NC_UL_BIG:
              error_at(index_expr->location(), "index value overflow");
              return Expression::make_error(location);
+           default:
+             go_unreachable();
            }
 
          Named_type* ntype = Type::lookup_integer_type("int");
          Integer_type* inttype = ntype->integer_type();
-         mpz_t max;
-         mpz_init_set_ui(max, 1);
-         mpz_mul_2exp(max, max, inttype->bits() - 1);
-         bool ok = mpz_cmp(ival, max) < 0;
-         mpz_clear(max);
-         if (!ok)
+         if (sizeof(index) <= static_cast<size_t>(inttype->bits() * 8)
+             && index >> (inttype->bits() - 1) != 0)
            {
-             mpz_clear(ival);
              error_at(index_expr->location(), "index value overflow");
              return Expression::make_error(location);
            }
 
-         mpz_clear(ival);
-
-         // FIXME: Our representation isn't very good; this avoids
-         // thrashing.
-         if (index > 0x1000000)
-           {
-             error_at(index_expr->location(), "index too large for compiler");
-             return Expression::make_error(location);
-           }
-       }
-
-      if (index == vals.size())
-       vals.push_back(val);
-      else
-       {
-         if (index > vals.size())
+         if (std::find(indexes->begin(), indexes->end(), index)
+             != indexes->end())
            {
-             vals.reserve(index + 32);
-             vals.resize(index + 1, static_cast<Expression*>(NULL));
-           }
-         if (vals[index] != NULL)
-           {
-             error_at((index_expr != NULL
-                       ? index_expr->location()
-                       : val->location()),
-                      "duplicate value for index %lu",
+             error_at(index_expr->location(), "duplicate value for index %lu",
                       index);
              return Expression::make_error(location);
            }
-         vals[index] = val;
+
+         if (!indexes->empty() && index < indexes->back())
+           indexes_out_of_order = true;
+
+         indexes->push_back(index);
        }
 
+      vals->push_back(val);
+
       ++index;
     }
 
-  size_t size = vals.size();
-  Expression_list* list = new Expression_list;
-  list->reserve(size);
-  for (size_t i = 0; i < size; ++i)
-    list->push_back(vals[i]);
+  if (indexes->empty())
+    {
+      delete indexes;
+      indexes = NULL;
+    }
+
+  if (indexes_out_of_order)
+    {
+      typedef std::vector<std::pair<unsigned long, Expression*> > V;
 
-  return this->make_array(type, list);
+      V v;
+      v.reserve(indexes->size());
+      std::vector<unsigned long>::const_iterator pi = indexes->begin();
+      for (Expression_list::const_iterator pe = vals->begin();
+          pe != vals->end();
+          ++pe, ++pi)
+       v.push_back(std::make_pair(*pi, *pe));
+
+      std::sort(v.begin(), v.end(), Index_value_compare());
+
+      delete indexes;
+      delete vals;
+      indexes = new std::vector<unsigned long>();
+      indexes->reserve(v.size());
+      vals = new Expression_list();
+      vals->reserve(v.size());
+
+      for (V::const_iterator p = v.begin(); p != v.end(); ++p)
+       {
+         indexes->push_back(p->first);
+         vals->push_back(p->second);
+       }
+    }
+
+  return this->make_array(type, indexes, vals);
 }
 
 // Actually build the array composite literal. This handles
 // [...]{...}.
 
 Expression*
-Composite_literal_expression::make_array(Type* type, Expression_list* vals)
+Composite_literal_expression::make_array(
+    Type* type,
+    const std::vector<unsigned long>* indexes,
+    Expression_list* vals)
 {
-  source_location location = this->location();
+  Location location = this->location();
   Array_type* at = type->array_type();
+
   if (at->length() != NULL && at->length()->is_nil_expression())
     {
-      size_t size = vals == NULL ? 0 : vals->size();
+      size_t size;
+      if (vals == NULL)
+       size = 0;
+      else if (indexes != NULL)
+       size = indexes->back() + 1;
+      else
+       {
+         size = vals->size();
+         Integer_type* it = Type::lookup_integer_type("int")->integer_type();
+         if (sizeof(size) <= static_cast<size_t>(it->bits() * 8)
+             && size >> (it->bits() - 1) != 0)
+           {
+             error_at(location, "too many elements in composite literal");
+             return Expression::make_error(location);
+           }
+       }
+
       mpz_t vlen;
       mpz_init_set_ui(vlen, size);
       Expression* elen = Expression::make_integer(&vlen, NULL, location);
@@ -13047,10 +12731,43 @@ Composite_literal_expression::make_array(Type* type, Expression_list* vals)
       at = Type::make_array_type(at->element_type(), elen);
       type = at;
     }
+  else if (at->length() != NULL
+          && !at->length()->is_error_expression()
+          && this->vals_ != NULL)
+    {
+      Numeric_constant nc;
+      unsigned long val;
+      if (at->length()->numeric_constant_value(&nc)
+         && nc.to_unsigned_long(&val) == Numeric_constant::NC_UL_VALID)
+       {
+         if (indexes == NULL)
+           {
+             if (this->vals_->size() > val)
+               {
+                 error_at(location, "too many elements in composite literal");
+                 return Expression::make_error(location);
+               }
+           }
+         else
+           {
+             unsigned long max = indexes->back();
+             if (max >= val)
+               {
+                 error_at(location,
+                          ("some element keys in composite literal "
+                           "are out of range"));
+                 return Expression::make_error(location);
+               }
+           }
+       }
+    }
+
   if (at->length() != NULL)
-    return new Fixed_array_construction_expression(type, vals, location);
+    return new Fixed_array_construction_expression(type, indexes, vals,
+                                                  location);
   else
-    return new Open_array_construction_expression(type, vals, location);
+    return new Open_array_construction_expression(type, indexes, vals,
+                                                 location);
 }
 
 // Lower a map composite literal.
@@ -13060,7 +12777,7 @@ Composite_literal_expression::lower_map(Gogo* gogo, Named_object* function,
                                        Statement_inserter* inserter,
                                        Type* type)
 {
-  source_location location = this->location();
+  Location location = this->location();
   if (this->vals_ != NULL)
     {
       if (!this->has_keys_)
@@ -13114,7 +12831,7 @@ Composite_literal_expression::do_dump_expression(
 Expression*
 Expression::make_composite_literal(Type* type, int depth, bool has_keys,
                                   Expression_list* vals,
-                                  source_location location)
+                                  Location location)
 {
   return new Composite_literal_expression(type, depth, has_keys, vals,
                                          location);
@@ -13298,7 +13015,7 @@ Type_guard_expression::do_dump_expression(Ast_dump_context* ast_dump_context)
 
 Expression*
 Expression::make_type_guard(Expression* expr, Type* type,
-                           source_location location)
+                           Location location)
 {
   return new Type_guard_expression(expr, type, location);
 }
@@ -13311,7 +13028,7 @@ Expression::make_type_guard(Expression* expr, Type* type,
 class Heap_composite_expression : public Expression
 {
  public:
-  Heap_composite_expression(Expression* expr, source_location location)
+  Heap_composite_expression(Expression* expr, Location location)
     : Expression(EXPRESSION_HEAP_COMPOSITE, location),
       expr_(expr)
   { }
@@ -13359,7 +13076,7 @@ tree
 Heap_composite_expression::do_get_tree(Translate_context* context)
 {
   tree expr_tree = this->expr_->get_tree(context);
-  if (expr_tree == error_mark_node)
+  if (expr_tree == error_mark_node || TREE_TYPE(expr_tree) == error_mark_node)
     return error_mark_node;
   tree expr_size = TYPE_SIZE_UNIT(TREE_TYPE(expr_tree));
   go_assert(TREE_CODE(expr_size) == INTEGER_CST);
@@ -13367,12 +13084,13 @@ Heap_composite_expression::do_get_tree(Translate_context* context)
                                                expr_size, this->location());
   space = fold_convert(build_pointer_type(TREE_TYPE(expr_tree)), space);
   space = save_expr(space);
-  tree ref = build_fold_indirect_ref_loc(this->location(), space);
+  tree ref = build_fold_indirect_ref_loc(this->location().gcc_location(),
+                                         space);
   TREE_THIS_NOTRAP(ref) = 1;
   tree ret = build2(COMPOUND_EXPR, TREE_TYPE(space),
                    build2(MODIFY_EXPR, void_type_node, ref, expr_tree),
                    space);
-  SET_EXPR_LOCATION(ret, this->location());
+  SET_EXPR_LOCATION(ret, this->location().gcc_location());
   return ret;
 }
 
@@ -13390,7 +13108,7 @@ Heap_composite_expression::do_dump_expression(
 // Allocate a composite literal on the heap.
 
 Expression*
-Expression::make_heap_composite(Expression* expr, source_location location)
+Expression::make_heap_composite(Expression* expr, Location location)
 {
   return new Heap_composite_expression(expr, location);
 }
@@ -13436,12 +13154,18 @@ Receive_expression::do_check_types(Gogo*)
 tree
 Receive_expression::do_get_tree(Translate_context* context)
 {
+  Location loc = this->location();
+
   Channel_type* channel_type = this->channel_->type()->channel_type();
   if (channel_type == NULL)
     {
       go_assert(this->channel_->type()->is_error());
       return error_mark_node;
     }
+
+  Expression* td = Expression::make_type_descriptor(channel_type, loc);
+  tree td_tree = td->get_tree(context);
+
   Type* element_type = channel_type->element_type();
   Btype* element_type_btype = element_type->get_backend(context->gogo());
   tree element_type_tree = type_to_tree(element_type_btype);
@@ -13450,8 +13174,7 @@ Receive_expression::do_get_tree(Translate_context* context)
   if (element_type_tree == error_mark_node || channel == error_mark_node)
     return error_mark_node;
 
-  return Gogo::receive_from_channel(element_type_tree, channel,
-                                   this->for_select_, this->location());
+  return Gogo::receive_from_channel(element_type_tree, td_tree, channel, loc);
 }
 
 // Dump ast representation for a receive expression.
@@ -13466,7 +13189,7 @@ Receive_expression::do_dump_expression(Ast_dump_context* ast_dump_context) const
 // Make a receive expression.
 
 Receive_expression*
-Expression::make_receive(Expression* channel, source_location location)
+Expression::make_receive(Expression* channel, Location location)
 {
   return new Receive_expression(channel, location);
 }
@@ -13477,7 +13200,7 @@ Expression::make_receive(Expression* channel, source_location location)
 class Type_descriptor_expression : public Expression
 {
  public:
-  Type_descriptor_expression(Type* type, source_location location)
+  Type_descriptor_expression(Type* type, Location location)
     : Expression(EXPRESSION_TYPE_DESCRIPTOR, location),
       type_(type)
   { }
@@ -13522,7 +13245,7 @@ Type_descriptor_expression::do_dump_expression(
 // Make a type descriptor expression.
 
 Expression*
-Expression::make_type_descriptor(Type* type, source_location location)
+Expression::make_type_descriptor(Type* type, Location location)
 {
   return new Type_descriptor_expression(type, location);
 }
@@ -13537,7 +13260,7 @@ class Type_info_expression : public Expression
 {
  public:
   Type_info_expression(Type* type, Type_info type_info)
-    : Expression(EXPRESSION_TYPE_INFO, BUILTINS_LOCATION),
+    : Expression(EXPRESSION_TYPE_INFO, Linemap::predeclared_location()),
       type_(type), type_info_(type_info)
   { }
 
@@ -13589,25 +13312,26 @@ Type_info_expression::do_type()
 tree
 Type_info_expression::do_get_tree(Translate_context* context)
 {
-  tree type_tree = type_to_tree(this->type_->get_backend(context->gogo()));
-  if (type_tree == error_mark_node)
-    return error_mark_node;
-
-  tree val_type_tree = type_to_tree(this->type()->get_backend(context->gogo()));
-  go_assert(val_type_tree != error_mark_node);
-
-  if (this->type_info_ == TYPE_INFO_SIZE)
-    return fold_convert_loc(BUILTINS_LOCATION, val_type_tree,
-                           TYPE_SIZE_UNIT(type_tree));
-  else
+  Btype* btype = this->type_->get_backend(context->gogo());
+  Gogo* gogo = context->gogo();
+  size_t val;
+  switch (this->type_info_)
     {
-      unsigned int val;
-      if (this->type_info_ == TYPE_INFO_ALIGNMENT)
-       val = go_type_alignment(type_tree);
-      else
-       val = go_field_alignment(type_tree);
-      return build_int_cstu(val_type_tree, val);
+    case TYPE_INFO_SIZE:
+      val = gogo->backend()->type_size(btype);
+      break;
+    case TYPE_INFO_ALIGNMENT:
+      val = gogo->backend()->type_alignment(btype);
+      break;
+    case TYPE_INFO_FIELD_ALIGNMENT:
+      val = gogo->backend()->type_field_alignment(btype);
+      break;
+    default:
+      go_unreachable();
     }
+  tree val_type_tree = type_to_tree(this->type()->get_backend(gogo));
+  go_assert(val_type_tree != error_mark_node);
+  return build_int_cstu(val_type_tree, val);
 }
 
 // Dump ast representation for a type info expression.
@@ -13643,7 +13367,8 @@ class Struct_field_offset_expression : public Expression
 {
  public:
   Struct_field_offset_expression(Struct_type* type, const Struct_field* field)
-    : Expression(EXPRESSION_STRUCT_FIELD_OFFSET, BUILTINS_LOCATION),
+    : Expression(EXPRESSION_STRUCT_FIELD_OFFSET,
+                Linemap::predeclared_location()),
       type_(type), field_(field)
   { }
 
@@ -13731,7 +13456,7 @@ Expression::make_struct_field_offset(Struct_type* type,
 class Map_descriptor_expression : public Expression
 {
  public:
-  Map_descriptor_expression(Map_type* type, source_location location)
+  Map_descriptor_expression(Map_type* type, Location location)
     : Expression(EXPRESSION_MAP_DESCRIPTOR, location),
       type_(type)
   { }
@@ -13778,7 +13503,7 @@ Map_descriptor_expression::do_dump_expression(
 // Make a map descriptor expression.
 
 Expression*
-Expression::make_map_descriptor(Map_type* type, source_location location)
+Expression::make_map_descriptor(Map_type* type, Location location)
 {
   return new Map_descriptor_expression(type, location);
 }
@@ -13788,7 +13513,7 @@ Expression::make_map_descriptor(Map_type* type, source_location location)
 class Label_addr_expression : public Expression
 {
  public:
-  Label_addr_expression(Label* label, source_location location)
+  Label_addr_expression(Label* label, Location location)
     : Expression(EXPRESSION_LABEL_ADDR, location),
       label_(label)
   { }
@@ -13824,7 +13549,7 @@ class Label_addr_expression : public Expression
 // Make an expression for the address of an unnamed label.
 
 Expression*
-Expression::make_label_addr(Label* label, source_location location)
+Expression::make_label_addr(Label* label, Location location)
 {
   return new Label_addr_expression(label, location);
 }
@@ -13913,3 +13638,600 @@ Expression_list::contains_error() const
       return true;
   return false;
 }
+
+// Class Numeric_constant.
+
+// Destructor.
+
+Numeric_constant::~Numeric_constant()
+{
+  this->clear();
+}
+
+// Copy constructor.
+
+Numeric_constant::Numeric_constant(const Numeric_constant& a)
+  : classification_(a.classification_), type_(a.type_)
+{
+  switch (a.classification_)
+    {
+    case NC_INVALID:
+      break;
+    case NC_INT:
+    case NC_RUNE:
+      mpz_init_set(this->u_.int_val, a.u_.int_val);
+      break;
+    case NC_FLOAT:
+      mpfr_init_set(this->u_.float_val, a.u_.float_val, GMP_RNDN);
+      break;
+    case NC_COMPLEX:
+      mpfr_init_set(this->u_.complex_val.real, a.u_.complex_val.real,
+                   GMP_RNDN);
+      mpfr_init_set(this->u_.complex_val.imag, a.u_.complex_val.imag,
+                   GMP_RNDN);
+      break;
+    default:
+      go_unreachable();
+    }
+}
+
+// Assignment operator.
+
+Numeric_constant&
+Numeric_constant::operator=(const Numeric_constant& a)
+{
+  this->clear();
+  this->classification_ = a.classification_;
+  this->type_ = a.type_;
+  switch (a.classification_)
+    {
+    case NC_INVALID:
+      break;
+    case NC_INT:
+    case NC_RUNE:
+      mpz_init_set(this->u_.int_val, a.u_.int_val);
+      break;
+    case NC_FLOAT:
+      mpfr_init_set(this->u_.float_val, a.u_.float_val, GMP_RNDN);
+      break;
+    case NC_COMPLEX:
+      mpfr_init_set(this->u_.complex_val.real, a.u_.complex_val.real,
+                   GMP_RNDN);
+      mpfr_init_set(this->u_.complex_val.imag, a.u_.complex_val.imag,
+                   GMP_RNDN);
+      break;
+    default:
+      go_unreachable();
+    }
+  return *this;
+}
+
+// Clear the contents.
+
+void
+Numeric_constant::clear()
+{
+  switch (this->classification_)
+    {
+    case NC_INVALID:
+      break;
+    case NC_INT:
+    case NC_RUNE:
+      mpz_clear(this->u_.int_val);
+      break;
+    case NC_FLOAT:
+      mpfr_clear(this->u_.float_val);
+      break;
+    case NC_COMPLEX:
+      mpfr_clear(this->u_.complex_val.real);
+      mpfr_clear(this->u_.complex_val.imag);
+      break;
+    default:
+      go_unreachable();
+    }
+  this->classification_ = NC_INVALID;
+}
+
+// Set to an unsigned long value.
+
+void
+Numeric_constant::set_unsigned_long(Type* type, unsigned long val)
+{
+  this->clear();
+  this->classification_ = NC_INT;
+  this->type_ = type;
+  mpz_init_set_ui(this->u_.int_val, val);
+}
+
+// Set to an integer value.
+
+void
+Numeric_constant::set_int(Type* type, const mpz_t val)
+{
+  this->clear();
+  this->classification_ = NC_INT;
+  this->type_ = type;
+  mpz_init_set(this->u_.int_val, val);
+}
+
+// Set to a rune value.
+
+void
+Numeric_constant::set_rune(Type* type, const mpz_t val)
+{
+  this->clear();
+  this->classification_ = NC_RUNE;
+  this->type_ = type;
+  mpz_init_set(this->u_.int_val, val);
+}
+
+// Set to a floating point value.
+
+void
+Numeric_constant::set_float(Type* type, const mpfr_t val)
+{
+  this->clear();
+  this->classification_ = NC_FLOAT;
+  this->type_ = type;
+  // Numeric constants do not have negative zero values, so remove
+  // them here.  They also don't have infinity or NaN values, but we
+  // should never see them here.
+  if (mpfr_zero_p(val))
+    mpfr_init_set_ui(this->u_.float_val, 0, GMP_RNDN);
+  else
+    mpfr_init_set(this->u_.float_val, val, GMP_RNDN);
+}
+
+// Set to a complex value.
+
+void
+Numeric_constant::set_complex(Type* type, const mpfr_t real, const mpfr_t imag)
+{
+  this->clear();
+  this->classification_ = NC_COMPLEX;
+  this->type_ = type;
+  mpfr_init_set(this->u_.complex_val.real, real, GMP_RNDN);
+  mpfr_init_set(this->u_.complex_val.imag, imag, GMP_RNDN);
+}
+
+// Get an int value.
+
+void
+Numeric_constant::get_int(mpz_t* val) const
+{
+  go_assert(this->is_int());
+  mpz_init_set(*val, this->u_.int_val);
+}
+
+// Get a rune value.
+
+void
+Numeric_constant::get_rune(mpz_t* val) const
+{
+  go_assert(this->is_rune());
+  mpz_init_set(*val, this->u_.int_val);
+}
+
+// Get a floating point value.
+
+void
+Numeric_constant::get_float(mpfr_t* val) const
+{
+  go_assert(this->is_float());
+  mpfr_init_set(*val, this->u_.float_val, GMP_RNDN);
+}
+
+// Get a complex value.
+
+void
+Numeric_constant::get_complex(mpfr_t* real, mpfr_t* imag) const
+{
+  go_assert(this->is_complex());
+  mpfr_init_set(*real, this->u_.complex_val.real, GMP_RNDN);
+  mpfr_init_set(*imag, this->u_.complex_val.imag, GMP_RNDN);
+}
+
+// Express value as unsigned long if possible.
+
+Numeric_constant::To_unsigned_long
+Numeric_constant::to_unsigned_long(unsigned long* val) const
+{
+  switch (this->classification_)
+    {
+    case NC_INT:
+    case NC_RUNE:
+      return this->mpz_to_unsigned_long(this->u_.int_val, val);
+    case NC_FLOAT:
+      return this->mpfr_to_unsigned_long(this->u_.float_val, val);
+    case NC_COMPLEX:
+      if (!mpfr_zero_p(this->u_.complex_val.imag))
+       return NC_UL_NOTINT;
+      return this->mpfr_to_unsigned_long(this->u_.complex_val.real, val);
+    default:
+      go_unreachable();
+    }
+}
+
+// Express integer value as unsigned long if possible.
+
+Numeric_constant::To_unsigned_long
+Numeric_constant::mpz_to_unsigned_long(const mpz_t ival,
+                                      unsigned long *val) const
+{
+  if (mpz_sgn(ival) < 0)
+    return NC_UL_NEGATIVE;
+  unsigned long ui = mpz_get_ui(ival);
+  if (mpz_cmp_ui(ival, ui) != 0)
+    return NC_UL_BIG;
+  *val = ui;
+  return NC_UL_VALID;
+}
+
+// Express floating point value as unsigned long if possible.
+
+Numeric_constant::To_unsigned_long
+Numeric_constant::mpfr_to_unsigned_long(const mpfr_t fval,
+                                       unsigned long *val) const
+{
+  if (!mpfr_integer_p(fval))
+    return NC_UL_NOTINT;
+  mpz_t ival;
+  mpz_init(ival);
+  mpfr_get_z(ival, fval, GMP_RNDN);
+  To_unsigned_long ret = this->mpz_to_unsigned_long(ival, val);
+  mpz_clear(ival);
+  return ret;
+}
+
+// Convert value to integer if possible.
+
+bool
+Numeric_constant::to_int(mpz_t* val) const
+{
+  switch (this->classification_)
+    {
+    case NC_INT:
+    case NC_RUNE:
+      mpz_init_set(*val, this->u_.int_val);
+      return true;
+    case NC_FLOAT:
+      if (!mpfr_integer_p(this->u_.float_val))
+       return false;
+      mpz_init(*val);
+      mpfr_get_z(*val, this->u_.float_val, GMP_RNDN);
+      return true;
+    case NC_COMPLEX:
+      if (!mpfr_zero_p(this->u_.complex_val.imag)
+         || !mpfr_integer_p(this->u_.complex_val.real))
+       return false;
+      mpz_init(*val);
+      mpfr_get_z(*val, this->u_.complex_val.real, GMP_RNDN);
+      return true;
+    default:
+      go_unreachable();
+    }
+}
+
+// Convert value to floating point if possible.
+
+bool
+Numeric_constant::to_float(mpfr_t* val) const
+{
+  switch (this->classification_)
+    {
+    case NC_INT:
+    case NC_RUNE:
+      mpfr_init_set_z(*val, this->u_.int_val, GMP_RNDN);
+      return true;
+    case NC_FLOAT:
+      mpfr_init_set(*val, this->u_.float_val, GMP_RNDN);
+      return true;
+    case NC_COMPLEX:
+      if (!mpfr_zero_p(this->u_.complex_val.imag))
+       return false;
+      mpfr_init_set(*val, this->u_.complex_val.real, GMP_RNDN);
+      return true;
+    default:
+      go_unreachable();
+    }
+}
+
+// Convert value to complex.
+
+bool
+Numeric_constant::to_complex(mpfr_t* vr, mpfr_t* vi) const
+{
+  switch (this->classification_)
+    {
+    case NC_INT:
+    case NC_RUNE:
+      mpfr_init_set_z(*vr, this->u_.int_val, GMP_RNDN);
+      mpfr_init_set_ui(*vi, 0, GMP_RNDN);
+      return true;
+    case NC_FLOAT:
+      mpfr_init_set(*vr, this->u_.float_val, GMP_RNDN);
+      mpfr_init_set_ui(*vi, 0, GMP_RNDN);
+      return true;
+    case NC_COMPLEX:
+      mpfr_init_set(*vr, this->u_.complex_val.real, GMP_RNDN);
+      mpfr_init_set(*vi, this->u_.complex_val.imag, GMP_RNDN);
+      return true;
+    default:
+      go_unreachable();
+    }
+}
+
+// Get the type.
+
+Type*
+Numeric_constant::type() const
+{
+  if (this->type_ != NULL)
+    return this->type_;
+  switch (this->classification_)
+    {
+    case NC_INT:
+      return Type::make_abstract_integer_type();
+    case NC_RUNE:
+      return Type::make_abstract_character_type();
+    case NC_FLOAT:
+      return Type::make_abstract_float_type();
+    case NC_COMPLEX:
+      return Type::make_abstract_complex_type();
+    default:
+      go_unreachable();
+    }
+}
+
+// If the constant can be expressed in TYPE, then set the type of the
+// constant to TYPE and return true.  Otherwise return false, and, if
+// ISSUE_ERROR is true, report an appropriate error message.
+
+bool
+Numeric_constant::set_type(Type* type, bool issue_error, Location loc)
+{
+  bool ret;
+  if (type == NULL)
+    ret = true;
+  else if (type->integer_type() != NULL)
+    ret = this->check_int_type(type->integer_type(), issue_error, loc);
+  else if (type->float_type() != NULL)
+    ret = this->check_float_type(type->float_type(), issue_error, loc);
+  else if (type->complex_type() != NULL)
+    ret = this->check_complex_type(type->complex_type(), issue_error, loc);
+  else
+    go_unreachable();
+  if (ret)
+    this->type_ = type;
+  return ret;
+}
+
+// Check whether the constant can be expressed in an integer type.
+
+bool
+Numeric_constant::check_int_type(Integer_type* type, bool issue_error,
+                                Location location) const
+{
+  mpz_t val;
+  switch (this->classification_)
+    {
+    case NC_INT:
+    case NC_RUNE:
+      mpz_init_set(val, this->u_.int_val);
+      break;
+
+    case NC_FLOAT:
+      if (!mpfr_integer_p(this->u_.float_val))
+       {
+         if (issue_error)
+           error_at(location, "floating point constant truncated to integer");
+         return false;
+       }
+      mpz_init(val);
+      mpfr_get_z(val, this->u_.float_val, GMP_RNDN);
+      break;
+
+    case NC_COMPLEX:
+      if (!mpfr_integer_p(this->u_.complex_val.real)
+         || !mpfr_zero_p(this->u_.complex_val.imag))
+       {
+         if (issue_error)
+           error_at(location, "complex constant truncated to integer");
+         return false;
+       }
+      mpz_init(val);
+      mpfr_get_z(val, this->u_.complex_val.real, GMP_RNDN);
+      break;
+
+    default:
+      go_unreachable();
+    }
+
+  bool ret;
+  if (type->is_abstract())
+    ret = true;
+  else
+    {
+      int bits = mpz_sizeinbase(val, 2);
+      if (type->is_unsigned())
+       {
+         // For an unsigned type we can only accept a nonnegative
+         // number, and we must be able to represents at least BITS.
+         ret = mpz_sgn(val) >= 0 && bits <= type->bits();
+       }
+      else
+       {
+         // For a signed type we need an extra bit to indicate the
+         // sign.  We have to handle the most negative integer
+         // specially.
+         ret = (bits + 1 <= type->bits()
+                || (bits <= type->bits()
+                    && mpz_sgn(val) < 0
+                    && (mpz_scan1(val, 0)
+                        == static_cast<unsigned long>(type->bits() - 1))
+                    && mpz_scan0(val, type->bits()) == ULONG_MAX));
+       }
+    }
+
+  if (!ret && issue_error)
+    error_at(location, "integer constant overflow");
+
+  return ret;
+}
+
+// Check whether the constant can be expressed in a floating point
+// type.
+
+bool
+Numeric_constant::check_float_type(Float_type* type, bool issue_error,
+                                  Location location) const
+{
+  mpfr_t val;
+  switch (this->classification_)
+    {
+    case NC_INT:
+    case NC_RUNE:
+      mpfr_init_set_z(val, this->u_.int_val, GMP_RNDN);
+      break;
+
+    case NC_FLOAT:
+      mpfr_init_set(val, this->u_.float_val, GMP_RNDN);
+      break;
+
+    case NC_COMPLEX:
+      if (!mpfr_zero_p(this->u_.complex_val.imag))
+       {
+         if (issue_error)
+           error_at(location, "complex constant truncated to float");
+         return false;
+       }
+      mpfr_init_set(val, this->u_.complex_val.real, GMP_RNDN);
+      break;
+
+    default:
+      go_unreachable();
+    }
+
+  bool ret;
+  if (type->is_abstract())
+    ret = true;
+  else if (mpfr_nan_p(val) || mpfr_inf_p(val) || mpfr_zero_p(val))
+    {
+      // A NaN or Infinity always fits in the range of the type.
+      ret = true;
+    }
+  else
+    {
+      mp_exp_t exp = mpfr_get_exp(val);
+      mp_exp_t max_exp;
+      switch (type->bits())
+       {
+       case 32:
+         max_exp = 128;
+         break;
+       case 64:
+         max_exp = 1024;
+         break;
+       default:
+         go_unreachable();
+       }
+
+      ret = exp <= max_exp;
+    }
+
+  mpfr_clear(val);
+
+  if (!ret && issue_error)
+    error_at(location, "floating point constant overflow");
+
+  return ret;
+} 
+
+// Check whether the constant can be expressed in a complex type.
+
+bool
+Numeric_constant::check_complex_type(Complex_type* type, bool issue_error,
+                                    Location location) const
+{
+  if (type->is_abstract())
+    return true;
+
+  mp_exp_t max_exp;
+  switch (type->bits())
+    {
+    case 64:
+      max_exp = 128;
+      break;
+    case 128:
+      max_exp = 1024;
+      break;
+    default:
+      go_unreachable();
+    }
+
+  mpfr_t real;
+  switch (this->classification_)
+    {
+    case NC_INT:
+    case NC_RUNE:
+      mpfr_init_set_z(real, this->u_.int_val, GMP_RNDN);
+      break;
+
+    case NC_FLOAT:
+      mpfr_init_set(real, this->u_.float_val, GMP_RNDN);
+      break;
+
+    case NC_COMPLEX:
+      if (!mpfr_nan_p(this->u_.complex_val.imag)
+         && !mpfr_inf_p(this->u_.complex_val.imag)
+         && !mpfr_zero_p(this->u_.complex_val.imag))
+       {
+         if (mpfr_get_exp(this->u_.complex_val.imag) > max_exp)
+           {
+             if (issue_error)
+               error_at(location, "complex imaginary part overflow");
+             return false;
+           }
+       }
+      mpfr_init_set(real, this->u_.complex_val.real, GMP_RNDN);
+      break;
+
+    default:
+      go_unreachable();
+    }
+
+  bool ret;
+  if (mpfr_nan_p(real) || mpfr_inf_p(real) || mpfr_zero_p(real))
+    ret = true;
+  else
+    ret = mpfr_get_exp(real) <= max_exp;
+
+  mpfr_clear(real);
+
+  if (!ret && issue_error)
+    error_at(location, "complex real part overflow");
+
+  return ret;
+}
+
+// Return an Expression for this value.
+
+Expression*
+Numeric_constant::expression(Location loc) const
+{
+  switch (this->classification_)
+    {
+    case NC_INT:
+      return Expression::make_integer(&this->u_.int_val, this->type_, loc);
+    case NC_RUNE:
+      return Expression::make_character(&this->u_.int_val, this->type_, loc);
+    case NC_FLOAT:
+      return Expression::make_float(&this->u_.float_val, this->type_, loc);
+    case NC_COMPLEX:
+      return Expression::make_complex(&this->u_.complex_val.real,
+                                     &this->u_.complex_val.imag,
+                                     this->type_, loc);
+    default:
+      go_unreachable();
+    }
+}