Skip to content

Add a flag to disable the traversing subterm analysis - #22540

Open
thomas-lamiaux wants to merge 5 commits into
rocq-prover:masterfrom
thomas-lamiaux:guard_flags
Open

thomas-lamiaux wants to merge 5 commits into
rocq-prover:masterfrom
thomas-lamiaux:guard_flags

Conversation

@thomas-lamiaux

@thomas-lamiaux thomas-lamiaux commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

Add a flag to disable the traversing subterm analysis

Fixes / closes #????

  • Added / updated test-suite.
  • Added changelog.
  • Added / updated documentation.
    • Documented any new / changed user messages.

@thomas-lamiaux
thomas-lamiaux requested review from a team as code owners September 28, 2026 15:20
@coqbot-app coqbot-app Bot added 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 Sep 28, 2026
@thomas-lamiaux

Copy link
Copy Markdown
Contributor Author

@yannl35133 can you review the change to inductive.ml ?

@thomas-lamiaux

Copy link
Copy Markdown
Contributor Author

No idea where to fix the checker issue

@SkySkimmer

Copy link
Copy Markdown
Contributor

checker/values.ml
Ask me to do it if too annoying

@yannl35133

Copy link
Copy Markdown
Contributor

If you define a fixpoint with subterm analysis, and then unset it and use the definition featuring this fixpoint, unfolding it won't type check any more. I don't know how to solve this. The same issue exists with Unset Guard Checking, but at least the standard library doesn't define unguarded fixpoints.
Also, these options should probably be merged with the global guard flag.

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

Labels

needs: full CI The latest GitLab pipeline that ran was a light CI. Say "@coqbot run full ci" to get a full CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants