Skip to content

perf: remove obsolete old codegen workaround #9311

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

perf: remove obsolete old codegen workaround #9311

wants to merge 1 commit into from

Conversation

Kha
Copy link
Member

@Kha Kha commented Jul 11, 2025

No description provided.

@Kha Kha added the awaiting-mathlib We should not merge this until we have a successful Mathlib build label Jul 11, 2025
@Kha
Copy link
Member Author

Kha commented Jul 11, 2025

!bench

@leanprover-bot
Copy link
Collaborator

Here are the benchmark results for commit 9a91298.
The entire run failed.
Found no significant differences.

@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 Jul 11, 2025
@leanprover-bot
Copy link
Collaborator

leanprover-bot commented Jul 11, 2025 •
edited
Loading

Reference manual CI status:

  • ❗ Reference manual CI can not be attempted yet, as the nightly-testing-2025-07-10 tag does not exist there yet. We will retry when you push more commits. If you rebase your branch onto nightly-with-manual, reference manual CI should run now. You can force reference manual CI using the force-manual-ci label. (2025-07-11 15:47:26)
  • ❗ Reference manual CI will not be attempted unless your PR branches off the nightly-with-manual branch. Try git rebase 5778a3c0f2dbe17fdac88902478362d66c43fd61 --onto d3dda9f6d4428a906c096067ecb75e432afc4615. You can force reference manual CI using the force-manual-ci label. (2025-07-14 13:50:27)

leanprover-community-mathlib4-bot added a commit to leanprover-community/batteries that referenced this pull request Jul 11, 2025
@leanprover-community-mathlib4-bot
Update lean-toolchain for testing leanprover/lean4#9311
c0bf938
leanprover-community-mathlib4-bot added a commit to leanprover-community/mathlib4-nightly-testing that referenced this pull request Jul 11, 2025
@leanprover-community-mathlib4-bot
Update lean-toolchain for testing leanprover/lean4#9311
39a4e8d
@leanprover-community-bot leanprover-community-bot added builds-mathlib CI has verified that Mathlib builds against this PR and removed awaiting-mathlib We should not merge this until we have a successful Mathlib build labels Jul 11, 2025
@leanprover-community-bot
Copy link
Collaborator

leanprover-community-bot commented Jul 11, 2025 •
edited
Loading

  • ✅ Mathlib branch lean-pr-testing-9311 has successfully built against this PR. (2025-07-11 16:47:59) View Log
  • ❗ Batteries/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase 5778a3c0f2dbe17fdac88902478362d66c43fd61 --onto 18a82c04fc59d3dba392308e09d65ee431240d89. You can force Mathlib CI using the force-mathlib-ci label. (2025-07-14 13:50:26)

@zwarich
Copy link
Collaborator

zwarich commented Jul 11, 2025

!bench

@leanprover-bot
Copy link
Collaborator

Here are the benchmark results for commit 9a91298.
The entire run failed.
Found no significant differences.

@Kha Kha force-pushed the push-rquwqnkrswpu branch from 9a91298 to 62a615f Compare July 14, 2025 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
builds-mathlib CI has verified that Mathlib builds against this PR toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Kha @leanprover-bot @leanprover-community-bot @zwarich