Skip to content

[Merged by Bors] - fix: make List.rec and Nat.rec computable#1720

Closed
eric-wieser wants to merge 9 commits intomasterfrom
eric-wieser/List.rec-computable
Closed

[Merged by Bors] - fix: make List.rec and Nat.rec computable#1720
eric-wieser wants to merge 9 commits intomasterfrom
eric-wieser/List.rec-computable

Conversation

@eric-wieser
Copy link
Copy Markdown
Member

@eric-wieser eric-wieser commented Jan 20, 2023

This works around leanprover/lean4#2049. By manually adding compiler support for these recursors, we make a large number of porting notes redundant.


Open in Gitpod

@eric-wieser eric-wieser added the WIP Work in progress label Jan 20, 2023
@eric-wieser eric-wieser changed the title fix: make list.rec computable fix: make List.rec and Nat.rec computable Jan 20, 2023
@eric-wieser eric-wieser added awaiting-review awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. and removed WIP Work in progress labels Jan 20, 2023
Copy link
Copy Markdown
Member

@jcommelin jcommelin left a comment

Choose a reason for hiding this comment

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

Thanks 🎉

bors merge

@kim-em kim-em added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Jan 20, 2023
bors bot pushed a commit that referenced this pull request Jan 20, 2023
This works around leanprover/lean4#2049. By manually adding compiler support for these recursors, we make a large number of porting notes redundant.
@bors
Copy link
Copy Markdown

bors bot commented Jan 20, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title fix: make List.rec and Nat.rec computable [Merged by Bors] - fix: make List.rec and Nat.rec computable Jan 20, 2023
@bors bors bot closed this Jan 20, 2023
@bors bors bot deleted the eric-wieser/List.rec-computable branch January 20, 2023 18:06
mathlib-bors bot pushed a commit that referenced this pull request Nov 19, 2023
#1720 made `List.rec` computable and therefore made it possible to restore the simpler versions of
`foldrRecOn` and `foldlRecOn`.
alexkeizer pushed a commit that referenced this pull request Nov 21, 2023
#1720 made `List.rec` computable and therefore made it possible to restore the simpler versions of
`foldrRecOn` and `foldlRecOn`.
grunweg pushed a commit that referenced this pull request Dec 15, 2023
#1720 made `List.rec` computable and therefore made it possible to restore the simpler versions of
`foldrRecOn` and `foldlRecOn`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. ready-to-merge This PR has been sent to bors.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants