-
Notifications
You must be signed in to change notification settings - Fork 701
Separate unsynchronized and synchronized grammar states in procq #21348
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?
Conversation
|
@coqbot bench |
|
🏁 Bench results: INFO: failed to install rocq-mathcomp-boot (dependency rocq-elpi failed) 🐢 Top 25 slow downs┌─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┐ │ TOP 25 SLOW DOWNS │ │ │ │ OLD NEW DIFF %DIFF Ln FILE │ ├─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┤ │ 82.2 83.4 1.2698 1.55% 48 coq-fiat-crypto-with-bedrock/src/Curves/Weierstrass/AffineProofs.v.html │ │ 1.36 2.54 1.1795 86.69% 549 coq-fiat-crypto-with-bedrock/src/Bedrock/P256/Jacobian.v.html │ │ 236 237 0.9202 0.39% 141 coq-fiat-crypto-with-bedrock/src/UnsaturatedSolinasHeuristics/Tests.v.html │ │ 2.48 3.31 0.8298 33.43% 34 coq-fiat-crypto-with-bedrock/src/Rewriter/Passes/ArithWithCasts.v.html │ │ 3.86 4.59 0.7278 18.85% 128 coq-fiat-parsers/src/Parsers/Refinement/SharpenedJSON.v.html │ │ 0.287 0.892 0.6051 210.73% 12 rocq-stdlib/theories/MSets/MSets.v.html │ │ 1.90 2.47 0.5673 29.83% 32 coq-fiat-crypto-with-bedrock/src/Rewriter/Passes/NBE.v.html │ │ 133 134 0.5229 0.39% 155 coq-fiat-crypto-with-bedrock/src/UnsaturatedSolinasHeuristics/Tests.v.html │ │ 201 202 0.5216 0.26% 8 coq-neural-net-interp-computed-lite/theories/MaxOfTwoNumbersSimpler/Computed/AllLogits.v.html │ │ 26.2 26.7 0.4986 1.90% 62 coq-fiat-crypto-with-bedrock/src/Assembly/Parse/TestAsm.v.html │ │ 21.5 21.9 0.4755 2.22% 49 coq-fiat-crypto-with-bedrock/src/Curves/Weierstrass/AffineProofs.v.html │ │ 19.8 20.3 0.4727 2.39% 708 coq-fiat-crypto-with-bedrock/src/Rewriter/RulesProofs.v.html │ │ 64.4 64.9 0.4491 0.70% 716 coq-fiat-crypto-with-bedrock/src/Bedrock/Secp256k1/JacobianCoZ.v.html │ │ 173 174 0.4325 0.25% 233 coq-fiat-crypto-with-bedrock/rupicola/bedrock2/deps/riscv-coq/src/riscv/Proofs/DecodeByExtension.v.html │ │ 18.6 18.9 0.3278 1.76% 77 coq-fiat-crypto-with-bedrock/src/Assembly/Parse/TestAsm.v.html │ │ 0.260 0.554 0.2946 113.48% 19 rocq-stdlib/theories/FSets/FSetFacts.v.html │ │ 20.8 21.0 0.2851 1.37% 23 coq-fiat-crypto-with-bedrock/src/Rewriter/Passes/Arith.v.html │ │ 26.4 26.7 0.2758 1.05% 374 coq-unimath/UniMath/ModelCategories/Generated/LNWFSMonoidalStructure.v.html │ │ 0.270 0.544 0.2744 101.79% 12 rocq-stdlib/theories/ZArith/Znumtheory.v.html │ │ 0.163 0.428 0.2652 162.67% 596 rocq-stdlib/theories/Strings/Byte.v.html │ │ 1.15 1.41 0.2621 22.79% 207 rocq-stdlib/theories/setoid_ring/Ncring_tac.v.html │ │ 26.4 26.6 0.2611 0.99% 375 coq-unimath/UniMath/ModelCategories/Generated/LNWFSMonoidalStructure.v.html │ │ 0.278 0.537 0.2590 93.06% 11 rocq-stdlib/theories/Strings/HexString.v.html │ │ 34.4 34.6 0.2525 0.73% 174 coq-fiat-crypto-with-bedrock/src/Bedrock/P256/Jacobian.v.html │ │ 18.7 18.9 0.2524 1.35% 31 coq-engine-bench-lite/coq/PerformanceDemos/pattern.v.html │ └─────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘ 🐇 Top 25 speed ups┌────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┐ │ TOP 25 SPEED UPS │ │ │ │ OLD NEW DIFF %DIFF Ln FILE │ ├────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┤ │ 64.7 63.6 -1.0715 -1.66% 608 coq-bedrock2/bedrock2/src/bedrock2Examples/lightbulb.v.html │ │ 95.5 94.5 -0.9190 -0.96% 999 coq-performance-tests-lite/src/fiat_crypto_via_setoid_rewrite_standalone.v.html │ │ 118 118 -0.8925 -0.75% 22 coq-fiat-crypto-with-bedrock/src/Rewriter/Passes/ArithWithCasts.v.html │ │ 93.2 92.5 -0.7328 -0.79% 20 coq-fiat-crypto-with-bedrock/src/Rewriter/Passes/NBE.v.html │ │ 95.3 94.6 -0.6894 -0.72% 968 coq-performance-tests-lite/src/fiat_crypto_via_setoid_rewrite_standalone.v.html │ │ 45.6 45.0 -0.5781 -1.27% 3 coq-fiat-crypto-with-bedrock/src/ExtractionJsOfOCaml/WithBedrock/fiat_crypto.v.html │ │ 3.85 3.32 -0.5228 -13.60% 557 coq-fiat-crypto-with-bedrock/src/Bedrock/P256/Jacobian.v.html │ │ 2.05 1.53 -0.5221 -25.43% 313 rocq-stdlib/theories/Strings/Byte.v.html │ │ 27.5 27.0 -0.5029 -1.83% 68 coq-fiat-crypto-with-bedrock/rupicola/bedrock2/deps/riscv-coq/src/riscv/Proofs/VerifyDecode.v.html │ │ 3.21 2.71 -0.4988 -15.54% 607 rocq-stdlib/theories/Zmod/ZmodBase.v.html │ │ 0.895 0.474 -0.4208 -47.03% 249 rocq-stdlib/theories/Structures/OrdersEx.v.html │ │ 43.3 42.9 -0.3742 -0.86% 2 coq-fiat-crypto-with-bedrock/src/ExtractionJsOfOCaml/fiat_crypto.v.html │ │ 30.6 30.3 -0.3256 -1.06% 225 coq-fiat-crypto-with-bedrock/src/Bedrock/P256/Coord32.v.html │ │ 0.568 0.248 -0.3192 -56.23% 1 rocq-stdlib/theories/micromega/ZifyPow.v.html │ │ 0.551 0.242 -0.3098 -56.19% 11 rocq-stdlib/theories/Strings/OctalString.v.html │ │ 0.576 0.270 -0.3053 -53.03% 36 rocq-stdlib/theories/MSets/MSetAVL.v.html │ │ 32.279 31.983 -0.2960 -0.92% 97 coq-vst/veric/binop_lemmas5.v.html │ │ 0.467 0.172 -0.2944 -63.10% 13 rocq-stdlib/theories/micromega/ZCoeff.v.html │ │ 0.594 0.303 -0.2913 -49.00% 17 rocq-stdlib/theories/Logic/IndefiniteDescription.v.html │ │ 2.76 2.47 -0.2812 -10.20% 1001 coq-performance-tests-lite/src/fiat_crypto_via_setoid_rewrite_standalone.v.html │ │ 0.615 0.339 -0.2758 -44.83% 1161 rocq-stdlib/theories/Strings/Byte.v.html │ │ 8.19 7.92 -0.2709 -3.31% 1831 coq-fiat-crypto-with-bedrock/rupicola/bedrock2/compiler/src/compiler/FlatToRiscvFunctions.v.html │ │ 0.492 0.228 -0.2643 -53.68% 14 rocq-stdlib/theories/setoid_ring/Ring_polynom.v.html │ │ 0.504 0.241 -0.2631 -52.17% 11 rocq-stdlib/theories/ZArith/Zminmax.v.html │ │ 14.1 13.8 -0.2567 -1.82% 216 coq-fiat-crypto-with-bedrock/src/Fancy/Barrett256.v.html │ └────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘ |
The unsynchronized part is shared over the whole process, but in principle we could have multiple synchronized parsing states (eg with multicore, or a more functional style summary)
116cc25 to
ac53fd2
Compare
The unsynchronized part is shared over the whole process, but in
principle we could have multiple synchronized parsing states (eg with
multicore, or a more functional style summary)