aboutsummaryrefslogtreecommitdiff
path: root/src/check_stmt.cpp
diff options
context:
space:
mode:
authorgingerBill <bill@gingerbill.org>2018-10-13 11:07:56 +0100
committergingerBill <bill@gingerbill.org>2018-10-13 11:07:56 +0100
commit73e9dbbf8c4a68dc6c512eb2de568d59df046494 (patch)
treebb2806ad8b6b078ff3b901cd2e5dba857dc23f32 /src/check_stmt.cpp
parent0971a59493d601458ccb386a3752a75f6d880b8f (diff)
switch on typeid with type cases
Diffstat (limited to 'src/check_stmt.cpp')
-rw-r--r--src/check_stmt.cpp48
1 files changed, 31 insertions, 17 deletions
diff --git a/src/check_stmt.cpp b/src/check_stmt.cpp
index 94de84f86..0ae83f1ed 100644
--- a/src/check_stmt.cpp
+++ b/src/check_stmt.cpp
@@ -729,32 +729,46 @@ void check_switch_stmt(CheckerContext *ctx, Ast *node, u32 mod_flags) {
add_constant_switch_case(ctx, &seen, rhs);
} else {
Operand y = {};
- check_expr(ctx, &y, expr);
+ if (is_type_typeid(x.type)) {
+ check_expr_or_type(ctx, &y, expr, x.type);
+ } else {
+ check_expr(ctx, &y, expr);
+ }
if (x.mode == Addressing_Invalid ||
y.mode == Addressing_Invalid) {
continue;
}
- convert_to_typed(ctx, &y, x.type);
- if (y.mode == Addressing_Invalid) {
- continue;
- }
+ if (y.mode == Addressing_Type) {
+ Type *t = y.type;
+ if (t == nullptr || t == t_invalid || is_type_polymorphic(t)) {
+ error(y.expr, "Invalid type for case clause");
+ continue;
+ }
+ t = default_type(t);
+ add_type_info_type(ctx, t);
+ } else {
+ convert_to_typed(ctx, &y, x.type);
+ if (y.mode == Addressing_Invalid) {
+ continue;
+ }
- // NOTE(bill): the ordering here matters
- Operand z = y;
- check_comparison(ctx, &z, &x, Token_CmpEq);
- if (z.mode == Addressing_Invalid) {
- continue;
- }
- if (y.mode != Addressing_Constant) {
- if (complete) {
- error(y.expr, "#complete switch statement only allows constant case clauses");
+ // NOTE(bill): the ordering here matters
+ Operand z = y;
+ check_comparison(ctx, &z, &x, Token_CmpEq);
+ if (z.mode == Addressing_Invalid) {
+ continue;
+ }
+ if (y.mode != Addressing_Constant) {
+ if (complete) {
+ error(y.expr, "#complete switch statement only allows constant case clauses");
+ }
+ continue;
}
- continue;
- }
- add_constant_switch_case(ctx, &seen, y);
+ add_constant_switch_case(ctx, &seen, y);
+ }
}
}