Skip to content

[LLZK-384] fix llzk-inline-structs pass bug#216

Merged
tim-hoffman merged 13 commits intomainfrom
th/LLZK-384
Nov 17, 2025
Merged

[LLZK-384] fix llzk-inline-structs pass bug#216
tim-hoffman merged 13 commits intomainfrom
th/LLZK-384

Conversation

@tim-hoffman
Copy link
Member

@tim-hoffman tim-hoffman commented Nov 14, 2025

Fix a couple bugs in llzk-inline-structs pass revealed by a test case from zirgen frontend.

@tim-hoffman tim-hoffman force-pushed the th/LLZK-384 branch 2 times, most recently from 78a93fc to 92290e6 Compare November 14, 2025 21:29
@tim-hoffman tim-hoffman changed the base branch from main to th/refactor_inline_structs_pass November 15, 2025 04:10
Base automatically changed from th/refactor_inline_structs_pass to main November 17, 2025 17:34
@tim-hoffman tim-hoffman requested a review from a team November 17, 2025 17:43
Copy link
Contributor

@iangneal iangneal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just one nit

Co-authored-by: Ian Neal <ian@veridise.com>
@tim-hoffman tim-hoffman requested a review from a team November 17, 2025 18:52
@tim-hoffman tim-hoffman merged commit 31f3184 into main Nov 17, 2025
10 checks passed
@tim-hoffman tim-hoffman deleted the th/LLZK-384 branch November 17, 2025 19:30
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.

2 participants