Skip to content

perf: populate the Sym.simp grind normalization theorems at attribute time - #15725

Open
leodemoura wants to merge 1 commit into
masterfrom
grind_sym_norm_ext
Open

leodemoura wants to merge 1 commit into
masterfrom
grind_sym_norm_ext

Conversation

@leodemoura

Copy link
Copy Markdown
Member

This PR removes the rebuild of the Sym.simp normalization theorem set on every pattern normalization in grind. With backward.grind.normalizer := false, every [grind] attribute application and every theorem referenced by instantiate/finish paid a full conversion of the [grind norm] set (grind_ite_trace timed out).

The Sym.simp theorems now live in their own environment extension, normSymExt, populated by the [grind norm]/[grind unfold] attributes together with normExt; the normalizer only reads the extension state. Requires a stage0 update.

🤖 Generated with Claude Code

…bute time

This PR removes the rebuild of the `Sym.simp` normalization theorem set on every pattern normalization in `grind`. With `backward.grind.normalizer := false`, every `[grind]` attribute application and every theorem referenced by `instantiate`/`finish` paid a full conversion of the `[grind norm]` set (`grind_ite_trace` timed out).

The `Sym.simp` theorems now live in their own environment extension, `normSymExt`, populated by the `[grind norm]`/`[grind unfold]` attributes together with `normExt`; the normalizer only reads the extension state. Requires a stage0 update.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@leodemoura leodemoura added the changelog-tactics User facing tactics label Oct 16, 2026
@leodemoura
leodemoura enabled auto-merge October 16, 2026 03:16
@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 16, 2026

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

changelog-tactics User facing tactics 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.

1 participant