fix(codegen): evaluate and release len temporaries #62

Merged

rdenadai merged 1 commit from fix/codegen-len-temporary into main 2026-10-06 19:38:50 +00:00

Conversation 0 Commits 1 Files changed 4 +98 -10

rdenadai commented 2026-10-06 19:29:00 +00:00

Owner

Copy link

Summary

len now evaluates array expressions even when their length is known statically, preserving initializer side effects. It also releases temporary arrays, lists, and strings after reading their length while leaving borrowed values owned by their bindings.

Regression coverage checks array initializer side effects, borrowed array use after len, and release calls for temporary managed values. The built-ins example now demonstrates these behaviors.

Tests

• Ruff format and lint checks
• Full test suite: 2742 passed, 1 skipped, 35 xfailed
• Native built-ins tests: 100 passed

## Summary `len` now evaluates array expressions even when their length is known statically, preserving initializer side effects. It also releases temporary arrays, lists, and strings after reading their length while leaving borrowed values owned by their bindings. Regression coverage checks array initializer side effects, borrowed array use after `len`, and release calls for temporary managed values. The built-ins example now demonstrates these behaviors. ## Tests - Ruff format and lint checks - Full test suite: 2742 passed, 1 skipped, 35 xfailed - Native built-ins tests: 100 passed

rdenadai added 1 commit 2026-10-06 19:29:00 +00:00

fix(codegen): evaluate and release len temporaries ... 347a85633b

Preserve side effects when array length is known statically and release managed values created only for a length query.

rdenadai merged commit 02b84db144 into main 2026-10-06 19:38:50 +00:00

rdenadai referenced this pull request from a commit 2026-10-06 19:38:51 +00:00

Merge pull request 'fix(codegen): evaluate and release len temporaries' (#62) from fix/codegen-len-temporary into main

rdenadai referenced this pull request 2026-10-06 19:49:30 +00:00

feat(parser): implement ordered comparison chains #63

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

rdenadai/sigil!62

No description provided.