feat: detect unused constructor projection scratch (PS6140) - #1019
Merged
Merged
Conversation
Register opt-in PS6140 with source-bound public constructor, allocation geometry, concrete projection, immutable class, observation, retention and release proofs. Preserve required fallback buffers and keep native count, alias, finalizer and completion assumptions explicit. Add complete original Metal and CUDA source replays, conventional analyzer fixtures, parallel adversarial controls and scoped configuration documentation. No automatic edit is supplied. Related to issue 890.
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.
Summary
Evidence and coverage
Local validation
TestPS6140tests pass with and without the race detector, including the genuine Metal and CUDA constructor/backend matrices and the shared retained-list controls.Related work: issue 890. Issue status will be updated separately after merge and successful CI on the exact default-branch commit.
Follow-up CI runtime repair
-parallel=1in 612.179 seconds. Its PS6131 campaign took 375.34 seconds and the twenty CUDA source-class cases added about 119 seconds.c6b89193f8e8c0c4e95cd2ba376725f093a52ff8was pushed without bypasses.c6b89193f8e8c0c4e95cd2ba376725f093a52ff8. Its actual tested merge checkout wasf62027daf3b000c6ad36ad73de8bfb2a7536807d, with parentsb38ccb035a955915850a0e7f0bb99c091d64c0beand the final head. No unresolved review threads, review comments, or reviews were present at the final merge check. CI on the exact merged default-branch commit still gates issue closure.