Skip to content

perf: inline assorted declarations used in match_expr#13716

Draft
hargoniX wants to merge 4 commits into
masterfrom
hbv/inline_match_expr
Draft

perf: inline assorted declarations used in match_expr#13716
hargoniX wants to merge 4 commits into
masterfrom
hbv/inline_match_expr

Conversation

@hargoniX
Copy link
Copy Markdown
Contributor

@hargoniX hargoniX commented May 12, 2026

blocked on #9929

@hargoniX
Copy link
Copy Markdown
Contributor Author

!bench

@leanprover-radar
Copy link
Copy Markdown

leanprover-radar commented May 12, 2026

Benchmark results for 1ed80e9 against 8c82f9e are in. There are significant results. @hargoniX

  • 🟥 build//instructions: +36.8G (+0.32%)

Large changes (1🟥)

  • 🟥 build/module/Lean.Elab.Tactic.BVDecide.Frontend.Normalize.Simproc//instructions: +5.1G (+18.08%)

Medium changes (5🟥)

  • 🟥 build/module/Lean.Elab.Tactic.Do.Internal.VCGen.Solve//instructions: +1.5G (+11.51%) (reduced significance based on absolute threshold)
  • 🟥 build/module/Lean.Elab.Tactic.Omega.Frontend//instructions: +3.3G (+9.70%) (reduced significance based on absolute threshold)
  • 🟥 build/module/Lean.Meta.Tactic.FunInd//instructions: +3.0G (+4.91%) (reduced significance based on absolute threshold)
  • 🟥 build/module/Lean.Meta.Tactic.Grind.EMatch//instructions: +1.3G (+2.56%) (reduced significance based on absolute threshold)
  • 🟥 build/module/Lean.Meta.Tactic.Grind.Ext//instructions: +1.2G (+15.57%) (reduced significance based on absolute threshold)

Small changes (1✅, 94🟥)

Too many entries to display here. View the full report on radar instead.

@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 May 12, 2026
@mathlib-lean-pr-testing
Copy link
Copy Markdown

mathlib-lean-pr-testing Bot commented May 12, 2026

Mathlib CI status (docs):

  • ❗ Batteries/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase 8c82f9ed4ae09b165342b24a6461ccc097caae50 --onto c04a83a2e5d5718032819f2d124f2c949fab292f. You can force Mathlib CI using the force-mathlib-ci label. (2026-05-12 16:12:46)
  • ❗ Batteries/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase 8c82f9ed4ae09b165342b24a6461ccc097caae50 --onto ed0d50fcf0efa8c9a32e1ccb3603ce43edce6926. You can force Mathlib CI using the force-mathlib-ci label. (2026-05-13 13:35:59)

@leanprover-bot
Copy link
Copy Markdown
Collaborator

leanprover-bot commented May 12, 2026

Reference manual CI status:

  • ❗ Reference manual CI will not be attempted unless your PR branches off the nightly-with-manual branch. Try git rebase 8c82f9ed4ae09b165342b24a6461ccc097caae50 --onto c04a83a2e5d5718032819f2d124f2c949fab292f. You can force reference manual CI using the force-manual-ci label. (2026-05-12 16:12:48)
  • ❗ Reference manual CI will not be attempted unless your PR branches off the nightly-with-manual branch. Try git rebase 8c82f9ed4ae09b165342b24a6461ccc097caae50 --onto ed0d50fcf0efa8c9a32e1ccb3603ce43edce6926. You can force reference manual CI using the force-manual-ci label. (2026-05-13 13:36:00)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

3 participants