Skip to content

do not let loop variables leak stack every iteration - #660

Open
xermicus wants to merge 9 commits into
mainfrom
cl/alloca-at-entry
Open

xermicus wants to merge 9 commits into
mainfrom
cl/alloca-at-entry

Conversation

@xermicus

@xermicus xermicus commented Oct 2, 2026

Copy link
Copy Markdown
Member

build_alloca emits each Yul let as an alloca where it is declared, not in the function's entry block. LLVM detects and cleans up most of this bug, which is why we didn't notice it at all.

@xermicus
xermicus requested review from elle-j and jubnzv October 2, 2026 15:10
@xermicus

Copy link
Copy Markdown
Member Author

After paritytech/polkavm#425 is released and we bump PVM the CI should go green on this.

This branch has not been deployed

No deployments
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