Repository navigation
Conversation
Contributor
|
Thank you for this contribution. Closing as incorporated into the published historical construction, with attribution retained—not as a rejection. We rechecked the current head
The new central ledger separates incorporated work from pending review. Mathematical claims retain the limitations in the linked review. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Change and mathematical scope
joint_dual_reclaim_compiler.acquire()sorts the complete retired-register set on every acquisition. Maintain sorted slot buckets by current frame rank instead, preserving the exact descending-rank, ascending-slot selection order. When clearing raises a retired anchor's frame, move that anchor between buckets before its next acquisition.This addresses the compiler bottleneck identified in #154 and is independent of its concurrent-check and frame-algebra changes. No mathematical hypotheses, emitted words, profile counts, or bound change. The source manifest and derived certificate digests are regenerated through the existing authoring scripts; the complete source closure remains enforced.
On a DGX Spark with Python 3.12, the prototype reduced the two compiler runs from a mean combined 140.1 s to 44.6 s (two runs per variant and axis, reversed order). A separate full original
verify_joint_dual.pycomparison, including C++ compilation, regeneration, independent replay, profiles and composition, was 128.2 s → 49.7 s (one run per variant). That benchmark used PR165 at7518fed2688baf25c7c32bae32674f3334b517da; its compiler source is byte-identical to current maind1d6c070f5a8c684727ee7ec35d930f9ebfa9758. These timings are scoped to that verifier, not the whole repository.Prepared by Chafik Boukhalfa with substantial OpenAI Codex assistance. The PR57 compiler and existing contributor attribution are retained.
Validation
Validated committed tree
f1081fabbfadaaaaf7292a40d428dbbb865ce920on the Spark (Python 3.12.3):make joint-dual-verify: PASS, including the full original verifier and all 10 existing control tests.make verify-*groups: PASS in isolated snapshots of the same source. This includesmake paired-cube-verifyand every isolated unit-test module. The tested source bytes were compared with the published commit.verify-positiverun lacked the Boost development headers that CI installs. After supplying Ubuntu's Boost 1.83 headers in a local dependency directory, a fresh clone of the identical source passed that complete group. No source or tracked artifact changed during the rerun.git diff --check: PASS.The complete h=23 and h=25 serialized words and compiler receipts must reproduce exactly, including every input and dirty basis column in both orientations. Independent word replay, prepared transitions, original C++ CRT profiles, all 47 strict inequalities, seven margins and existing adverse controls remain enabled. The conditional bound remains
4764513337/10^14.