checker, cgen: fix if/match expr with continue or break in a branch (#18466)

This commit is contained in:
yuyi 2023-06-16 17:06:00 +08:00 committed by GitHub
parent d17f6f69cd
commit ac32d2a803
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
6 changed files with 24 additions and 3 deletions

View File

@ -414,7 +414,7 @@ fn (mut c Checker) if_expr(mut node ast.IfExpr) ast.Type {
node.pos) node.pos)
} }
} }
} else if !node.is_comptime && stmt !is ast.Return { } else if !node.is_comptime && stmt !in [ast.Return, ast.BranchStmt] {
c.error('`${if_kind}` expression requires an expression as the last statement of every branch', c.error('`${if_kind}` expression requires an expression as the last statement of every branch',
branch.pos) branch.pos)
} }

View File

@ -90,7 +90,7 @@ fn (mut c Checker) match_expr(mut node ast.MatchExpr) ast.Type {
c.check_match_branch_last_stmt(stmt, ret_type, expr_type) c.check_match_branch_last_stmt(stmt, ret_type, expr_type)
} }
} }
} else if stmt !is ast.Return { } else if stmt !in [ast.Return, ast.BranchStmt] {
if node.is_expr && ret_type != ast.void_type { if node.is_expr && ret_type != ast.void_type {
c.error('`match` expression requires an expression as the last statement of every branch', c.error('`match` expression requires an expression as the last statement of every branch',
stmt.pos) stmt.pos)

View File

@ -1802,7 +1802,7 @@ fn (mut g Gen) stmts_with_tmp_var(stmts []ast.Stmt, tmp_var string) bool {
g.set_current_pos_as_last_stmt_pos() g.set_current_pos_as_last_stmt_pos()
g.skip_stmt_pos = true g.skip_stmt_pos = true
mut is_noreturn := false mut is_noreturn := false
if stmt is ast.Return { if stmt in [ast.Return, ast.BranchStmt] {
is_noreturn = true is_noreturn = true
} else if stmt is ast.ExprStmt { } else if stmt is ast.ExprStmt {
is_noreturn = is_noreturn_callexpr(stmt.expr) is_noreturn = is_noreturn_callexpr(stmt.expr)

View File

@ -25,6 +25,8 @@ fn (mut g Gen) need_tmp_var_in_if(node ast.IfExpr) bool {
} }
} else if branch.stmts[0] is ast.Return { } else if branch.stmts[0] is ast.Return {
return true return true
} else if branch.stmts[0] is ast.BranchStmt {
return true
} }
} }
} }

View File

@ -29,6 +29,8 @@ fn (mut g Gen) need_tmp_var_in_match(node ast.MatchExpr) bool {
} }
} else if branch.stmts[0] is ast.Return { } else if branch.stmts[0] is ast.Return {
return true return true
} else if branch.stmts[0] is ast.BranchStmt {
return true
} }
} }
} }

View File

@ -0,0 +1,17 @@
fn test_if_expr_with_continue_in_branch() {
mut result := []int{}
for i in 0 .. 10 {
s := if i < 2 {
100 + i
} else {
continue
}
println(s)
result << s
}
println(result)
assert result.len == 2
assert result[0] == 100
assert result[1] == 101
}