feat(lowering): materialize global rule array locals - #45
Closed
Teakowa wants to merge 3 commits into
Closed
Conversation
Extend deterministic rule-local global storage to nested arrays of scalar WIR values and local array foreach collections, while failing closed for object/reference and re-entrant semantics. Refs #31
Assert exact local-array foreach action ordering, generated helper identities, provenance, and an unsupported vector-array diagnostic. Refs #31
Verify exact source spans for materialized array locals and foreach collection/index helpers. Refs #31
Contributor
Author
|
Closed as superseded by #56. |
This was referenced Aug 24, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
foreachslice to read a materialized local array collectionHI018for player-context, object/reference, and re-entrant storageReview order
This slice is stacked on PR #44 (
agent/del-rs-31-foreach-20260818). Merge order is #41, #43, #44, then this PR.Evidence
cargo test --locked --test workshop_lowering -- --nocapture: 20 passedcargo test --locked --all-targets: passedcargo test --locked --test matrix --test corpus: 5 passedcargo run --locked --quiet -- matrix --check: 128 entries validwhileorder, canonical WIR Array/GlobalVariable/valueInArray forms, generated helper names, and provenanceHI018git diff --checkand task-file rustfmt checks passBoundaries
Refs #31