Skip to content

Account for redex which unlocks neutral terms in guard condition reduction failures#22142

Open
yannl35133 wants to merge 2 commits into
rocq-prover:masterfrom
Yann-Leray:needreduce-neutrals
Open

Account for redex which unlocks neutral terms in guard condition reduction failures#22142
yannl35133 wants to merge 2 commits into
rocq-prover:masterfrom
Yann-Leray:needreduce-neutrals

Conversation

@yannl35133

Copy link
Copy Markdown
Contributor

Fixes / closes #22141

@yannl35133 yannl35133 added the kind: fix This fixes a bug or incorrect documentation. label Jun 18, 2026
@yannl35133 yannl35133 requested a review from a team as a code owner June 18, 2026 17:45
@yannl35133 yannl35133 added needs: full CI The latest GitLab pipeline that ran was a light CI. Say "@coqbot run full ci" to get a full CI. part: fixpoints About Fixpoint, fix and mutual statements labels Jun 18, 2026
@yannl35133

Copy link
Copy Markdown
Contributor Author

@coqbot run full ci

@coqbot-app coqbot-app Bot removed the needs: full CI The latest GitLab pipeline that ran was a light CI. Say "@coqbot run full ci" to get a full CI. label Jun 18, 2026
@ppedrot ppedrot self-assigned this Jun 18, 2026
@yannl35133 yannl35133 added needs: discussion Further discussion is needed. request: full CI Use this label when you want your next push to trigger a full CI. labels Jun 19, 2026
@yannl35133 yannl35133 force-pushed the needreduce-neutrals branch from 9a7205d to b4a9ebf Compare June 19, 2026 13:47
@coqbot-app coqbot-app Bot removed the request: full CI Use this label when you want your next push to trigger a full CI. label Jun 19, 2026
@yannl35133

Copy link
Copy Markdown
Contributor Author

Some tests in Fixpoint.v actually break SN, but others are also refused by this PR even though they don't break SN. I also don't know what was the issue for equations_test and category-theory.
We probably need to discuss what we want the specification to be and how to consider the failing tests.

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

Labels

kind: fix This fixes a bug or incorrect documentation. needs: discussion Further discussion is needed. part: fixpoints About Fixpoint, fix and mutual statements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SN failure: subterm erased even though neutral head could unlock it

2 participants