Fix FIrrelevant conversion in RR#19414
Conversation
6dbb97d to
c8264d6
Compare
c8264d6 to
704d5ae
Compare
704d5ae to
6313ceb
Compare
|
@coqbot run full ci |
6313ceb to
0cc47b6
Compare
|
The "needs: rebase" label was set more than 30 days ago. If the PR is not rebased in 30 days, it will be automatically closed. |
0cc47b6 to
91524bd
Compare
91524bd to
53678c5
Compare
|
The "needs: rebase" label was set more than 30 days ago. If the PR is not rebased in 30 days, it will be automatically closed. |
745e6f6 to
09a1000
Compare
09a1000 to
8df8790
Compare
|
@coqbot bench |
|
🏁 Bench results: INFO: failed to install rocq-mathcomp-algebra (dependency rocq-mathcomp-fingroup failed) 🐢 Top 25 slow downs┌────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┐ │ TOP 25 SLOW DOWNS │ │ │ │ OLD NEW DIFF %DIFF Ln FILE │ ├────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┤ │ 5.53 7.18 1.6465 29.78% 217 coq-fiat-crypto-with-bedrock/src/Bedrock/Field/Synthesis/Examples/p224_64_new.v.html │ │ 236 238 1.5103 0.64% 141 coq-fiat-crypto-with-bedrock/src/UnsaturatedSolinasHeuristics/Tests.v.html │ │ 55.2 56.5 1.2696 2.30% 27 coq-fiat-crypto-with-bedrock/src/Rewriter/Passes/ToFancyWithCasts.v.html │ │ 107 108 1.1726 1.09% 22 coq-fiat-crypto-with-bedrock/src/Rewriter/Passes/ArithWithCasts.v.html │ │ 34.3 35.5 1.1674 3.40% 898 coq-fiat-crypto-with-bedrock/src/Bedrock/Secp256k1/JoyeLadder.v.html │ │ 17.7 18.8 1.0446 5.89% 31 coq-engine-bench-lite/coq/PerformanceDemos/pattern.v.html │ │ 36.7 37.7 1.0257 2.79% 139 coq-fiat-parsers/src/Parsers/Refinement/SharpenedJSON.v.html │ │ 31.1 32.1 0.9833 3.16% 180 coq-fiat-crypto-with-bedrock/src/Bedrock/Field/Synthesis/Examples/p224_64_new.v.html │ │ 32.4 33.4 0.9826 3.03% 121 coq-fiat-crypto-with-bedrock/src/Bedrock/Field/Synthesis/Examples/p224_64_new.v.html │ │ 31.2 32.1 0.9490 3.04% 157 coq-fiat-crypto-with-bedrock/src/Bedrock/Field/Synthesis/Examples/p224_64_new.v.html │ │ 31.3 32.2 0.8754 2.80% 166 coq-fiat-crypto-with-bedrock/src/Bedrock/Field/Synthesis/Examples/p224_64_new.v.html │ │ 31.3 32.2 0.8678 2.77% 139 coq-fiat-crypto-with-bedrock/src/Bedrock/Field/Synthesis/Examples/p224_64_new.v.html │ │ 31.3 32.1 0.8488 2.71% 198 coq-fiat-crypto-with-bedrock/src/Bedrock/Field/Synthesis/Examples/p224_64_new.v.html │ │ 31.4 32.2 0.8329 2.65% 148 coq-fiat-crypto-with-bedrock/src/Bedrock/Field/Synthesis/Examples/p224_64_new.v.html │ │ 2.46 3.29 0.8310 33.83% 34 coq-fiat-crypto-with-bedrock/src/Rewriter/Passes/ArithWithCasts.v.html │ │ 26.4 27.2 0.7920 3.00% 794 coq-fiat-crypto-with-bedrock/src/Bedrock/Secp256k1/JoyeLadder.v.html │ │ 31.3 32.1 0.7907 2.53% 214 coq-fiat-crypto-with-bedrock/src/Bedrock/Field/Synthesis/Examples/p224_64_new.v.html │ │ 133 134 0.7434 0.56% 155 coq-fiat-crypto-with-bedrock/src/UnsaturatedSolinasHeuristics/Tests.v.html │ │ 26.9 27.6 0.7191 2.67% 148 coq-fiat-crypto-with-bedrock/src/Bedrock/End2End/X25519/GarageDoorTop.v.html │ │ 26.6 27.3 0.7093 2.67% 34 coq-fiat-crypto-with-bedrock/src/Bedrock/End2End/X25519/MontgomeryLadderRISCV.v.html │ │ 28.7 29.4 0.6516 2.27% 145 coq-fiat-crypto-with-bedrock/src/Bedrock/End2End/X25519/GarageDoorTop.v.html │ │ 15.3 16.0 0.6475 4.23% 898 coq-fiat-crypto-with-bedrock/src/Bedrock/Secp256k1/JoyeLadder.v.html │ │ 28.5 29.1 0.5745 2.02% 31 coq-fiat-crypto-with-bedrock/src/Bedrock/End2End/X25519/MontgomeryLadderRISCV.v.html │ │ 15.0 15.6 0.5595 3.73% 841 coq-fiat-crypto-with-bedrock/src/Curves/Weierstrass/Jacobian/CoZ.v.html │ │ 0.827 1.34 0.5104 61.70% 408 rocq-stdlib/theories/MSets/MSetAVL.v.html │ └────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘ 🐇 Top 25 speed ups┌────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┐ │ TOP 25 SPEED UPS │ │ │ │ OLD NEW DIFF %DIFF Ln FILE │ ├────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┤ │ 94.2 93.1 -1.1021 -1.17% 968 coq-performance-tests-lite/src/fiat_crypto_via_setoid_rewrite_standalone.v.html │ │ 84.4 83.5 -0.9178 -1.09% 48 coq-fiat-crypto-with-bedrock/src/Curves/Weierstrass/AffineProofs.v.html │ │ 51.2 50.5 -0.6937 -1.36% 376 coq-unimath/UniMath/ModelCategories/Generated/LNWFSMonoidalStructure.v.html │ │ 56.7 56.1 -0.6267 -1.11% 516 coq-fiat-crypto-with-bedrock/src/Bedrock/End2End/X25519/EdwardsXYZT.v.html │ │ 22.3 21.6 -0.6230 -2.80% 49 coq-fiat-crypto-with-bedrock/src/Curves/Weierstrass/AffineProofs.v.html │ │ 93.6 93.1 -0.5279 -0.56% 999 coq-performance-tests-lite/src/fiat_crypto_via_setoid_rewrite_standalone.v.html │ │ 10.7 10.3 -0.4132 -3.85% 743 coq-fiat-crypto-with-bedrock/src/Bedrock/Secp256k1/JoyeLadder.v.html │ │ 0.656 0.283 -0.3730 -56.88% 374 rocq-stdlib/theories/Sorting/SetoidList.v.html │ │ 64.5 64.1 -0.3513 -0.54% 608 coq-fiat-crypto-with-bedrock/rupicola/bedrock2/bedrock2/src/bedrock2Examples/lightbulb.v.html │ │ 9.13 8.78 -0.3451 -3.78% 453 coq-unimath/UniMath/SyntheticHomotopyTheory/Circle2.v.html │ │ 201 200 -0.3280 -0.16% 8 coq-neural-net-interp-computed-lite/theories/MaxOfTwoNumbersSimpler/Computed/AllLogits.v.html │ │ 22.3 22.0 -0.2831 -1.27% 520 coq-fiat-crypto-with-bedrock/src/Bedrock/End2End/X25519/EdwardsXYZT.v.html │ │ 24.5 24.2 -0.2748 -1.12% 345 coq-fiat-crypto-with-bedrock/src/Curves/Montgomery/XZProofs.v.html │ │ 22.0 21.8 -0.2613 -1.18% 338 coq-unimath/UniMath/ModelCategories/Generated/LNWFSMonoidalStructure.v.html │ │ 0.570 0.310 -0.2596 -45.58% 14 rocq-stdlib/theories/ZArith/auxiliary.v.html │ │ 0.551 0.296 -0.2553 -46.34% 14 rocq-stdlib/theories/setoid_ring/Ring_polynom.v.html │ │ 35.9 35.6 -0.2538 -0.71% 195 coq-fiat-crypto-with-bedrock/src/Bedrock/P256/Jacobian.v.html │ │ 1.32 1.07 -0.2530 -19.17% 89 coq-engine-bench-lite/coq/PerformanceDemos/quadratic_reduction.v.html │ │ 0.521 0.270 -0.2513 -48.22% 12 rocq-stdlib/theories/ZArith/Znumtheory.v.html │ │ 0.716 0.465 -0.2512 -35.09% 59 rocq-stdlib/theories/ZArith/Zeuclid.v.html │ │ 13.7 13.4 -0.2493 -1.82% 930 coq-unimath/UniMath/CategoryTheory/Hyperdoctrines/PartialEqRels/ExponentialEqs.v.html │ │ 9.55 9.30 -0.2450 -2.57% 950 coq-unimath/UniMath/CategoryTheory/Hyperdoctrines/PartialEqRels/ExponentialEqs.v.html │ │ 0.547 0.303 -0.2437 -44.54% 19 rocq-stdlib/theories/ZArith/Int.v.html │ │ 0.480 0.252 -0.2273 -47.39% 707 rocq-stdlib/theories/MSets/MSetList.v.html │ │ 0.512 0.307 -0.2045 -39.98% 11 rocq-stdlib/theories/omega/OmegaLemmas.v.html │ └────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘ |
8df8790 to
4b34d7f
Compare
|
@coqbot bench |
|
🏁 Bench results: INFO: failed to install rocq-mathcomp-algebra (dependency rocq-mathcomp-fingroup failed) 🐢 Top 25 slow downs┌──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┐ │ TOP 25 SLOW DOWNS │ │ │ │ OLD NEW DIFF %DIFF Ln FILE │ ├──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┤ │ 54.2 59.6 5.4675 10.09% 296 coq-fiat-crypto-with-bedrock/src/Bedrock/Secp256k1/Addchain.v.html │ │ 63.5 67.4 3.8787 6.11% 608 coq-bedrock2/bedrock2/src/bedrock2Examples/lightbulb.v.html │ │ 30.8 32.5 1.7521 5.70% 214 coq-fiat-crypto-with-bedrock/src/Bedrock/Field/Synthesis/Examples/p224_64_new.v.html │ │ 30.9 32.5 1.5896 5.14% 148 coq-fiat-crypto-with-bedrock/src/Bedrock/Field/Synthesis/Examples/p224_64_new.v.html │ │ 30.8 32.3 1.5474 5.03% 166 coq-fiat-crypto-with-bedrock/src/Bedrock/Field/Synthesis/Examples/p224_64_new.v.html │ │ 30.9 32.4 1.5170 4.91% 180 coq-fiat-crypto-with-bedrock/src/Bedrock/Field/Synthesis/Examples/p224_64_new.v.html │ │ 31.0 32.5 1.5028 4.85% 139 coq-fiat-crypto-with-bedrock/src/Bedrock/Field/Synthesis/Examples/p224_64_new.v.html │ │ 30.9 32.4 1.4750 4.77% 157 coq-fiat-crypto-with-bedrock/src/Bedrock/Field/Synthesis/Examples/p224_64_new.v.html │ │ 32.1 33.6 1.4441 4.50% 121 coq-fiat-crypto-with-bedrock/src/Bedrock/Field/Synthesis/Examples/p224_64_new.v.html │ │ 30.9 32.3 1.3868 4.49% 198 coq-fiat-crypto-with-bedrock/src/Bedrock/Field/Synthesis/Examples/p224_64_new.v.html │ │ 82.4 83.5 1.1272 1.37% 48 coq-fiat-crypto-with-bedrock/src/Curves/Weierstrass/AffineProofs.v.html │ │ 0.401 1.42 1.0203 254.31% 547 coq-fiat-crypto-with-bedrock/src/Bedrock/P256/Jacobian.v.html │ │ 2.46 3.30 0.8335 33.85% 34 coq-fiat-crypto-with-bedrock/src/Rewriter/Passes/ArithWithCasts.v.html │ │ 48.4 49.3 0.8242 1.70% 376 coq-unimath/UniMath/ModelCategories/Generated/LNWFSMonoidalStructure.v.html │ │ 25.2 26.0 0.8200 3.25% 550 coq-fiat-crypto-with-bedrock/rupicola/bedrock2/bedrock2/src/bedrock2Examples/insertionsort.v.html │ │ 34.0 34.7 0.7210 2.12% 898 coq-fiat-crypto-with-bedrock/src/Bedrock/Secp256k1/JoyeLadder.v.html │ │ 42.3 43.0 0.7090 1.68% 221 coq-fiat-crypto-with-bedrock/src/Bedrock/P256/Coord32.v.html │ │ 36.4 37.0 0.6697 1.84% 222 coq-fiat-crypto-with-bedrock/src/Bedrock/P256/Coord32.v.html │ │ 24.2 24.9 0.6566 2.71% 550 coq-bedrock2/bedrock2/src/bedrock2Examples/insertionsort.v.html │ │ 14.8 15.5 0.6392 4.30% 841 coq-fiat-crypto-with-bedrock/src/Curves/Weierstrass/Jacobian/CoZ.v.html │ │ 0.892 1.52 0.6290 70.56% 572 rocq-stdlib/theories/MSets/MSetAVL.v.html │ │ 21.5 22.2 0.6216 2.89% 651 rocq-stdlib/theories/Zmod/ZmodBase.v.html │ │ 42.9 43.4 0.5467 1.27% 2 coq-fiat-crypto-with-bedrock/src/ExtractionJsOfOCaml/fiat_crypto.v.html │ │ 7.26 7.76 0.4985 6.86% 294 coq-fiat-crypto-with-bedrock/src/Bedrock/Secp256k1/Addchain.v.html │ │ 30.7 31.2 0.4796 1.56% 223 coq-fiat-crypto-with-bedrock/src/Bedrock/P256/Coord32.v.html │ └──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘ 🐇 Top 25 speed ups┌────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┐ │ TOP 25 SPEED UPS │ │ │ │ OLD NEW DIFF %DIFF Ln FILE │ ├────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┤ │ 7.22 5.53 -1.6896 -23.40% 217 coq-fiat-crypto-with-bedrock/src/Bedrock/Field/Synthesis/Examples/p224_64_new.v.html │ │ 95.1 93.7 -1.4209 -1.49% 968 coq-performance-tests-lite/src/fiat_crypto_via_setoid_rewrite_standalone.v.html │ │ 94.9 93.6 -1.2853 -1.35% 999 coq-performance-tests-lite/src/fiat_crypto_via_setoid_rewrite_standalone.v.html │ │ 1.99 0.986 -1.0087 -50.57% 546 coq-fiat-crypto-with-bedrock/src/Bedrock/P256/Jacobian.v.html │ │ 38.8 38.1 -0.7323 -1.89% 276 coq-fiat-crypto-with-bedrock/src/Bedrock/P256/Coord.v.html │ │ 43.9 43.2 -0.6970 -1.59% 578 coq-fiat-crypto-with-bedrock/rupicola/bedrock2/compiler/src/compiler/MMIO.v.html │ │ 51.8 51.2 -0.6425 -1.24% 571 coq-fiat-crypto-with-bedrock/src/Bedrock/End2End/X25519/EdwardsXYZT.v.html │ │ 2.45 1.89 -0.5658 -23.06% 32 coq-fiat-crypto-with-bedrock/src/Rewriter/Passes/NBE.v.html │ │ 46.7 46.2 -0.4964 -1.06% 278 coq-fiat-crypto-with-bedrock/src/Bedrock/P256/Jacobian.v.html │ │ 36.3 35.8 -0.4625 -1.27% 195 coq-fiat-crypto-with-bedrock/src/Bedrock/P256/Jacobian.v.html │ │ 8.14 7.77 -0.3739 -4.59% 1831 coq-fiat-crypto-with-bedrock/rupicola/bedrock2/compiler/src/compiler/FlatToRiscvFunctions.v.html │ │ 0.595 0.227 -0.3675 -61.78% 18 rocq-stdlib/theories/micromega/VarMap.v.html │ │ 44.3 43.9 -0.3482 -0.79% 257 coq-fiat-crypto-with-bedrock/src/Bedrock/P256/Jacobian.v.html │ │ 2.50 2.16 -0.3404 -13.59% 212 rocq-stdlib/theories/setoid_ring/Ncring_tac.v.html │ │ 0.689 0.368 -0.3207 -46.55% 374 rocq-stdlib/theories/Sorting/SetoidList.v.html │ │ 26.6 26.3 -0.3168 -1.19% 62 coq-fiat-crypto-with-bedrock/src/Assembly/Parse/TestAsm.v.html │ │ 14.2 13.9 -0.3165 -2.22% 216 coq-fiat-crypto-with-bedrock/src/Fancy/Barrett256.v.html │ │ 31.0 30.7 -0.3081 -0.99% 596 coq-fiat-crypto-with-bedrock/src/Bedrock/Secp256k1/JacobianCoZ.v.html │ │ 4.34 4.04 -0.3059 -7.04% 492 rocq-stdlib/theories/Reals/Cauchy/ConstructiveCauchyRealsMult.v.html │ │ 0.521 0.243 -0.2783 -53.43% 4 rocq-stdlib/theories/extraction/ExtrHaskellZInt.v.html │ │ 9.80 9.52 -0.2727 -2.78% 87 coq-engine-bench-lite/coq/PerformanceDemos/quadratic_reduction.v.html │ │ 38.6 38.3 -0.2645 -0.68% 224 coq-performance-tests-lite/PerformanceExperiments/rewrite_lift_lets_map.v.html │ │ 0.923 0.666 -0.2564 -27.79% 214 rocq-stdlib/theories/setoid_ring/Ncring_tac.v.html │ │ 44.8 44.5 -0.2517 -0.56% 3 coq-fiat-crypto-with-bedrock/src/ExtractionJsOfOCaml/bedrock2_fiat_crypto.v.html │ │ 59.1 58.8 -0.2452 -0.41% 659 coq-fiat-crypto-with-bedrock/src/Bedrock/Secp256k1/JacobianCoZ.v.html │ └────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘ |
… call term_of_fconstr
4b34d7f to
3ce4f06
Compare
|
@coqbot run full ci |
Adapt to rocq-prover/rocq#19414 (abstract CClosure.usubs)
Overlays: