compiler: remove Array_index_expression::is_lvalue_
As of CL 77510 it is never true.
Change-Id: I77eac8bd13701060ffe2a697f4e61838d8023f54
Reviewed-on: https://go-review.googlesource.com/c/gofrontend/+/394695
Reviewed-by: Than McIntosh <thanm@google.com>
Reviewed-by: Ian Lance Taylor <iant@google.com>
diff --git a/go/expressions.cc b/go/expressions.cc
index dce48e0..734ecb9 100644
--- a/go/expressions.cc
+++ b/go/expressions.cc
@@ -7671,8 +7671,7 @@
&& left_type->array_type()->length() == NULL)
{
Array_type* at = left_type->array_type();
- bool is_lvalue = false;
- left = at->get_value_pointer(context->gogo(), left, is_lvalue);
+ left = at->get_value_pointer(context->gogo(), left);
}
else if (left_type->interface_type() != NULL)
{
@@ -9276,7 +9275,7 @@
Type* unsafe_ptr_type = Type::make_pointer_type(Type::make_void_type());
Expression* a1 = Expression::make_type_descriptor(element_type, loc);
Expression* a2 = Expression::make_temporary_reference(s1tmp, loc);
- a2 = slice_type->array_type()->get_value_pointer(gogo, a2, false);
+ a2 = slice_type->array_type()->get_value_pointer(gogo, a2);
a2 = Expression::make_cast(unsafe_ptr_type, a2, loc);
Expression* a3 = Expression::make_temporary_reference(l1tmp, loc);
Expression* a4 = Expression::make_temporary_reference(c1tmp, loc);
@@ -13848,9 +13847,8 @@
}
else
{
- Expression* valptr =
- array_type->get_value_pointer(gogo, this->array_,
- this->is_lvalue_);
+ Expression* valptr = array_type->get_value_pointer(gogo,
+ this->array_);
Bexpression* ptr = valptr->get_backend(context);
ptr = gogo->backend()->pointer_offset_expression(ptr, start, loc);
@@ -13891,8 +13889,7 @@
Bexpression* offset = gogo->backend()->conditional_expression(bfn, int_btype,
cond, zero,
start, loc);
- Expression* valptr = array_type->get_value_pointer(gogo, this->array_,
- this->is_lvalue_);
+ Expression* valptr = array_type->get_value_pointer(gogo, this->array_);
Bexpression* val = valptr->get_backend(context);
val = gogo->backend()->pointer_offset_expression(val, offset, loc);
diff --git a/go/expressions.h b/go/expressions.h
index 92e8d8d..707c193 100644
--- a/go/expressions.h
+++ b/go/expressions.h
@@ -3055,7 +3055,7 @@
Expression* end, Expression* cap, Location location)
: Expression(EXPRESSION_ARRAY_INDEX, location),
array_(array), start_(start), end_(end), cap_(cap), type_(NULL),
- is_lvalue_(false), needs_bounds_check_(true), is_flattened_(false)
+ needs_bounds_check_(true), is_flattened_(false)
{ }
// Return the array.
@@ -3087,18 +3087,6 @@
end() const
{ return this->end_; }
- // Return whether this array index expression appears in an lvalue
- // (left hand side of assignment) context.
- bool
- is_lvalue() const
- { return this->is_lvalue_; }
-
- // Update this array index expression to indicate that it appears
- // in a left-hand-side or lvalue context.
- void
- set_is_lvalue()
- { this->is_lvalue_ = true; }
-
void
set_needs_bounds_check(bool b)
{ this->needs_bounds_check_ = b; }
@@ -3174,8 +3162,6 @@
Expression* cap_;
// The type of the expression.
Type* type_;
- // Whether expr appears in an lvalue context.
- bool is_lvalue_;
// Whether bounds check is needed.
bool needs_bounds_check_;
// Whether this has already been flattened.
diff --git a/go/types.cc b/go/types.cc
index 3de0bd3..ef65670 100644
--- a/go/types.cc
+++ b/go/types.cc
@@ -7815,7 +7815,7 @@
// Return an expression for a pointer to the values in ARRAY.
Expression*
-Array_type::get_value_pointer(Gogo*, Expression* array, bool is_lvalue) const
+Array_type::get_value_pointer(Gogo*, Expression* array) const
{
if (this->length() != NULL)
{
@@ -7828,25 +7828,6 @@
}
// Slice.
-
- if (is_lvalue)
- {
- Temporary_reference_expression* tref =
- array->temporary_reference_expression();
- Var_expression* ve = array->var_expression();
- if (tref != NULL)
- {
- tref = tref->copy()->temporary_reference_expression();
- tref->set_is_lvalue();
- array = tref;
- }
- else if (ve != NULL)
- {
- ve = new Var_expression(ve->named_object(), ve->location());
- array = ve;
- }
- }
-
return Expression::make_slice_info(array,
Expression::SLICE_INFO_VALUE_POINTER,
array->location());
diff --git a/go/types.h b/go/types.h
index c55345a..6d72e09 100644
--- a/go/types.h
+++ b/go/types.h
@@ -2800,7 +2800,7 @@
// Return an expression for the pointer to the values in an array.
Expression*
- get_value_pointer(Gogo*, Expression* array, bool is_lvalue) const;
+ get_value_pointer(Gogo*, Expression* array) const;
// Return an expression for the length of an array with this type.
Expression*