From 5cb047d108528cf8d0205dc4cdddf65963c9985a Mon Sep 17 00:00:00 2001 From: Ian Lance Taylor Date: Sat, 17 Sep 2011 00:11:27 +0000 Subject: [PATCH] Improve errors for invalid use of [...]type. From-SVN: r178921 --- gcc/go/gofrontend/expressions.cc | 2 +- gcc/go/gofrontend/parse.cc | 17 +++++++++++++++-- gcc/testsuite/go.test/test/ddd1.go | 5 ++++- 3 files changed, 20 insertions(+), 4 deletions(-) diff --git a/gcc/go/gofrontend/expressions.cc b/gcc/go/gofrontend/expressions.cc index 32f0612a1b0..70f5f7c4423 100644 --- a/gcc/go/gofrontend/expressions.cc +++ b/gcc/go/gofrontend/expressions.cc @@ -11789,7 +11789,7 @@ Array_construction_expression::do_check_types(Gogo*) } Expression* length = at->length(); - if (length != NULL) + if (length != NULL && !length->is_error_expression()) { mpz_t val; mpz_init(val); diff --git a/gcc/go/gofrontend/parse.cc b/gcc/go/gofrontend/parse.cc index 059d9646f13..b18c9892285 100644 --- a/gcc/go/gofrontend/parse.cc +++ b/gcc/go/gofrontend/parse.cc @@ -2761,8 +2761,21 @@ Parse::primary_expr(bool may_be_sink, bool may_be_composite_lit, else this->advance_token(); if (expr->is_error_expression()) - return expr; - ret = Expression::make_cast(ret->type(), expr, loc); + ret = expr; + else + { + Type* t = ret->type(); + if (t->classification() == Type::TYPE_ARRAY + && t->array_type()->length() != NULL + && t->array_type()->length()->is_nil_expression()) + { + error_at(ret->location(), + "invalid use of %<...%> in type conversion"); + ret = Expression::make_error(loc); + } + else + ret = Expression::make_cast(t, expr, loc); + } } } diff --git a/gcc/testsuite/go.test/test/ddd1.go b/gcc/testsuite/go.test/test/ddd1.go index ff6342843a9..83e32de7b6b 100644 --- a/gcc/testsuite/go.test/test/ddd1.go +++ b/gcc/testsuite/go.test/test/ddd1.go @@ -15,7 +15,7 @@ var ( _ = sum() _ = sum(1.0, 2.0) _ = sum(1.5) // ERROR "integer" - _ = sum("hello") // ERROR "convert|incompatible" + _ = sum("hello") // ERROR "string.*as type int|incompatible" _ = sum([]int{1}) // ERROR "slice literal.*as type int|incompatible" ) @@ -43,4 +43,7 @@ func bad(args ...int) { var x int _ = unsafe.Pointer(&x...) // ERROR "[.][.][.]" _ = unsafe.Sizeof(x...) // ERROR "[.][.][.]" + _ = [...]byte("foo") // ERROR "[.][.][.]" + _ = [...][...]int{{1,2,3},{4,5,6}} // ERROR "[.][.][.]" } +