Skip to content

Code to write to array indexes#552

Merged
jberthold merged 6 commits into
masterfrom
array-index-writes
May 1, 2025
Merged

Code to write to array indexes#552
jberthold merged 6 commits into
masterfrom
array-index-writes

Conversation

@jberthold

@jberthold jberthold commented May 1, 2025

Copy link
Copy Markdown
Collaborator

This completes the array support for kmir.

Closes #487

@jberthold jberthold marked this pull request as ready for review May 1, 2025 13:31
@ehildenb

ehildenb commented May 1, 2025

Copy link
Copy Markdown
Member

Can we add the same test to prove-rs test harness? You can run just those tests with make test-integration TEST_ARGS='-k test_prove_rs

@jberthold jberthold force-pushed the array-index-writes branch from 5129b6d to 9937a4c Compare May 1, 2025 13:54
@ehildenb

ehildenb commented May 1, 2025

Copy link
Copy Markdown
Member

We checked separately that the program was indeed present in the SMIR file and had not been optimized to the trivial program by the compiler.

@jberthold jberthold merged commit 226e29d into master May 1, 2025
6 checks passed
@jberthold jberthold deleted the array-index-writes branch May 1, 2025 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support for arrays

3 participants