diff options
| author | gingerBill <bill@gingerbill.org> | 2023-01-27 11:47:00 +0000 |
|---|---|---|
| committer | gingerBill <bill@gingerbill.org> | 2023-01-27 11:47:00 +0000 |
| commit | ccf4b48865f8d3c6f78f72cda5cfd5e46b108c02 (patch) | |
| tree | 01d633a6a33ea0169aba0e8ec91e66fe1cd6e2ba /src/llvm_backend_stmt.cpp | |
| parent | 96eae9410337854b1384d9868f7098603c9647c2 (diff) | |
Add extra checks for multiple assignments when emitting stores
Diffstat (limited to 'src/llvm_backend_stmt.cpp')
| -rw-r--r-- | src/llvm_backend_stmt.cpp | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/src/llvm_backend_stmt.cpp b/src/llvm_backend_stmt.cpp index c268ab09a..2284649e2 100644 --- a/src/llvm_backend_stmt.cpp +++ b/src/llvm_backend_stmt.cpp @@ -1555,12 +1555,25 @@ gb_internal void lb_build_assignment(lbProcedure *p, Array<lbAddr> &lvals, Slice lb_append_tuple_values(p, &inits, init); } + bool prev_in_assignment = p->in_multi_assignment; + + isize lval_count = 0; + for (lbAddr const &lval : lvals) { + if (lval.addr.value != nullptr) { + // check if it is not a blank identifier + lval_count += 1; + } + } + p->in_multi_assignment = lval_count > 1; + GB_ASSERT(lvals.count == inits.count); for_array(i, inits) { lbAddr lval = lvals[i]; lbValue init = inits[i]; lb_addr_store(p, lval, init); } + + p->in_multi_assignment = prev_in_assignment; } gb_internal void lb_build_return_stmt_internal(lbProcedure *p, lbValue res) { @@ -2047,6 +2060,7 @@ gb_internal void lb_build_assign_stmt(lbProcedure *p, AstAssignStmt *as) { lb_build_assignment(p, lvals, as->rhs); return; } + GB_ASSERT(as->lhs.count == 1); GB_ASSERT(as->rhs.count == 1); // NOTE(bill): Only 1 += 1 is allowed, no tuples |