Skip to content

fix: elaborate the do continuation only once when contracting pure () - #15682

Merged
sgraf812 merged 2 commits into
masterfrom
sg/issue-13858
Oct 19, 2026
Merged

sgraf812 merged 2 commits into
masterfrom
sg/issue-13858

Conversation

@sgraf812

@sgraf812 sgraf812 commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

This PR fixes mkBindUnlessPure to only run the continuation once, making sure that using pure () doesn't elaborate everything afterwards twice.

Closes #13858

@github-actions github-actions Bot added the toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN label Oct 14, 2026
@sgraf812
sgraf812 marked this pull request as ready for review October 14, 2026 08:19
@sgraf812 sgraf812 added the changelog-language Language features and metaprograms label Oct 14, 2026
@sgraf812
sgraf812 enabled auto-merge October 14, 2026 08:20
@sgraf812
sgraf812 disabled auto-merge October 14, 2026 08:20
@sgraf812 sgraf812 added the downstream Request a downstream-lean4 adaptation PR. label Oct 14, 2026
@downstream-lean4

Copy link
Copy Markdown

The adaptation PR for this PR is leanprover/downstream-lean4#167.

@sgraf812
sgraf812 added this pull request to the merge queue Oct 19, 2026
Merged via the queue into master with commit a946636 Oct 19, 2026
17 checks passed
@sgraf812
sgraf812 deleted the sg/issue-13858 branch October 19, 2026 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-language Language features and metaprograms downstream Request a downstream-lean4 adaptation PR. toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Exponential time growth when repeating pure ()

2 participants