Skip to content

chore: remove temporary decide simp lemmas now subsumed by core - #999

Open
attilavjda wants to merge 1 commit into
cedar-policy:mainfrom
attilavjda:chore/remove-temporary-decide-simp-lemmas
Open

chore: remove temporary decide simp lemmas now subsumed by core#999
attilavjda wants to merge 1 commit into
cedar-policy:mainfrom
attilavjda:chore/remove-temporary-decide-simp-lemmas

Conversation

@attilavjda

@attilavjda attilavjda commented Aug 10, 2026

Copy link
Copy Markdown

Issue #, if available:

Description of changes:

Cedar/Thm/Data/LT.lean defines four @[simp] lemmas in namespace Decide, with the comment

"Temporary simp lemmas. We should be able to get rid of these when we upgrade to a future version of Lean."

Batteries simpNF linter (lake exe runLinter Cedar) reports all four as redundant,

core simp proves each via decide_eq_true_eq, decide_eq_false_iff_not, decide_not, and Bool.not_eq_eq_eq_not,

no call sites, neither name is referenced anywhere,

lake build Cedar SymCC UnitTest SymTest is green with zero warnings.

Used Aristotle AI (Harmonic) in making this PR.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants