Files
carbon-lang/toolchain/check/handle_array.cpp
T
Richard Smith 76576c1387 Fix rejection of arrays with eight or nine elements. (#3199)
In general, LLVM's parsing of decimal integers to `APInt`s forms an
`APInt` that is 4n bits wide, where n is the length of the integer, and
the `isNegative` check only checks the high bit.

In this case, we form an `APInt` that is four bits wide, with the high
bit set, which we reject because we think it's "negative". These two
array lengths are the only ones where this happens -- if the decimal
integer value is two characters long, we form an `APInt` that is eight
bits wide but holds a value < 100, so the high bit is never set, and the
same applies for longer integers too.

An `IntegerLiteral` is never negative, so we don't need the `isNegative`
check, and in fact it only detects the n=8 and n=9 cases.
2023-09-06 21:32:17 +00:00

57 lines
2.1 KiB
C++

// Part of the Carbon Language project, under the Apache License v2.0 with LLVM
// Exceptions. See /LICENSE for license information.
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
#include "toolchain/check/context.h"
#include "toolchain/parse/node_kind.h"
#include "toolchain/sem_ir/node.h"
#include "toolchain/sem_ir/node_kind.h"
namespace Carbon::Check {
auto HandleArrayExpressionStart(Context& /*context*/,
Parse::Node /*parse_node*/) -> bool {
return true;
}
auto HandleArrayExpressionSemi(Context& context, Parse::Node parse_node)
-> bool {
context.node_stack().Push(parse_node);
return true;
}
auto HandleArrayExpression(Context& context, Parse::Node parse_node) -> bool {
// TODO: Handle array type with undefined bound.
if (context.parse_tree().node_kind(context.node_stack().PeekParseNode()) ==
Parse::NodeKind::ArrayExpressionSemi) {
context.node_stack().PopAndIgnore();
context.node_stack().PopAndIgnore();
return context.TODO(parse_node, "HandleArrayExpressionWithoutBounds");
}
auto bound_node_id = context.node_stack().PopExpression();
context.node_stack()
.PopAndDiscardSoloParseNode<Parse::NodeKind::ArrayExpressionSemi>();
auto element_type_node_id = context.node_stack().PopExpression();
auto bound_node = context.semantics_ir().GetNode(bound_node_id);
if (bound_node.kind() == SemIR::NodeKind::IntegerLiteral) {
auto bound_value = context.semantics_ir().GetIntegerLiteral(
bound_node.GetAsIntegerLiteral());
// TODO: Produce an error if the array type is too large.
if (bound_value.getBitWidth() <= 64) {
context.AddNodeAndPush(
parse_node,
SemIR::Node::ArrayType::Make(
parse_node, SemIR::TypeId::TypeType, bound_node_id,
context.ExpressionAsType(parse_node, element_type_node_id)));
return true;
}
}
CARBON_DIAGNOSTIC(InvalidArrayExpression, Error, "Invalid array expression.");
context.emitter().Emit(parse_node, InvalidArrayExpression);
context.node_stack().Push(parse_node, SemIR::NodeId::BuiltinError);
return true;
}
} // namespace Carbon::Check