-
Notifications
You must be signed in to change notification settings - Fork 693
Stop using auto with * in intuition solver. #21129
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Stop using auto with * in intuition solver. #21129
Conversation
This started to spit a warning from 8.17 onwards, it is probably time to get rid of this dubious behaviour. Fixes rocq-prover#4949: [intuition] should not use [auto with *] by default.
🔴 CI failures at commit c6cd7d6 without any failure in the test-suite ✔️ Corresponding jobs for the base commit 17eccdd succeeded ❔ Ask me to try to extract minimal test cases that can be added to the test-suite 🏃
|
Non trivial amount of broken code, which was not totally unexpected. I guess someone will have to write overlays... |
🔴 CI failures at commit c6cd7d6 without any failure in the test-suite ✔️ Corresponding jobs for the base commit 17eccdd succeeded ❔ Ask me to try to extract minimal test cases that can be added to the test-suite 🏃
|
🔴 CI failures at commit c6cd7d6 without any failure in the test-suite ✔️ Corresponding jobs for the base commit 17eccdd succeeded ❔ Ask me to try to extract minimal test cases that can be added to the test-suite 🏃
|
🔴 CI failures at commit c6cd7d6 without any failure in the test-suite ✔️ Corresponding jobs for the base commit 17eccdd succeeded ❔ Ask me to try to extract minimal test cases that can be added to the test-suite 🏃
|
🔴 CI failures at commit c6cd7d6 without any failure in the test-suite ✔️ Corresponding jobs for the base commit 17eccdd succeeded ❔ Ask me to try to extract minimal test cases that can be added to the test-suite 🏃
|
🔴 CI failures at commit c6cd7d6 without any failure in the test-suite ✔️ Corresponding jobs for the base commit 17eccdd succeeded ❔ Ask me to try to extract minimal test cases that can be added to the test-suite 🏃
|
🔴 CI failures at commit c6cd7d6 without any failure in the test-suite ✔️ Corresponding jobs for the base commit 17eccdd succeeded ❔ Ask me to try to extract minimal test cases that can be added to the test-suite 🏃
|
@JasonGross fcl will need a bbv bump it seems. |
🔴 CI failures at commit c6cd7d6 without any failure in the test-suite ✔️ Corresponding jobs for the base commit 17eccdd succeeded ❔ Ask me to try to extract minimal test cases that can be added to the test-suite 🏃
|
dependabot + CI takes care of this automatically, cf mit-plv/fiat-crypto#2168. Or is there a bbv bump beyond this? |
OK, then it should be fine. |
Fixing cross-crypto is a bit annoying. It depends on FCF which calls intuition auto with * a lot, and fixing the code takes some time. I've done a little bit already but if anybody wants to take the baton I wouldn't complain. |
How annoying the fix is to make really depends on how precise you want it to be. In principle you could add at the beginning of every file (* TODO stop using "with *" *)
Local Ltac intuition_solver ::= solve [auto with *]. and it should work. |
Meh. Out of procrastination I started writing a script to parse the warning messages and autofix the issues. |
Here is the fcf PR: adampetcher/fcf#50. It needs to be merged and then the corresponding submodule must be bumped in cross-crypto. |
This started to spit a warning from 8.17 onwards, it is probably time to get rid of this dubious behaviour.
Fixes #4949: [intuition] should not use [auto with *] by default.
Backwards compatible overlays: