-
-
Notifications
You must be signed in to change notification settings - Fork 165
fix(ci): restore gap budgets and validate GC probes and witnesses #11756
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
Merged
Merged
Changes from all commits
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
4fe05e2
fix(gc): refresh macOS arguments mapping leaf classification
proggeramlug 17f13fe
Add changeset for macOS arguments leaf table refresh
proggeramlug 4e043e9
Refresh Linux and Windows arguments mapping leaf tables from CI artif…
proggeramlug 1be2965
ci(gc): exercise Windows try roots under RS4GC
proggeramlug 443f9d0
changelog: name the Windows try-root probe repair for PR 11755
proggeramlug 05090dc
ci: halve full gap slices after twelve-way timeouts
proggeramlug 4c361d2
changelog: name the full-gap budget repair for PR 11756
proggeramlug 70d6d80
fix(ci): link macOS stdlib provider frameworks
proggeramlug 390936c
changelog: record macOS provider framework link repair
proggeramlug e6f5b7e
Merge commit '443f9d0b58d5428a7ea67ddf14ba7d6e64f52089' into codex/ma…
proggeramlug 5a779d7
fix(ci): export provider feature installer bootstrap symbols
proggeramlug 087b532
fix(ci): route immutable thread-global codegen tests
proggeramlug 684dd65
docs: add thread-global test routing changeset
proggeramlug 1e256b8
fix(ci): split fast PR gap slices after job timeout
proggeramlug 79affe7
merge: integrate main and immutable-global test routing
proggeramlug 3349e1b
Merge commit '420721032422900c04797dd6ca59155f54eb1295' into codex/11…
proggeramlug 61f29f1
fix(ci): route immutable-global tests after classifier refresh
proggeramlug 761608b
Merge commit '420721032422900c04797dd6ca59155f54eb1295' into codex/11…
proggeramlug 0ea5f9a
fix(ci): route immutable-global tests for provider validation
proggeramlug a8ea310
Merge commit '61f29f134e3ed6ae4638f4e88f74b016bb3bf476' into codex/ci…
proggeramlug 2f10898
Merge commit '0ea5f9afbef7e5d464cbda8cf2cb92da3d387c21' into codex/ci…
proggeramlug d863a28
fix(gc-effects): refresh Windows immutable-global helper classes
proggeramlug 6e6a66b
fix(ci): apply witness GC settings during compilation and execution
proggeramlug ec7117a
Merge origin/main into codex/full-gap-24-20261002
proggeramlug File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| Split the full auto-optimize gap suite into twenty-four shards after nine | ||
| of twelve workers hit the existing 110-minute limit. Split the fast PR | ||
| arm into twelve shards after its six-way shard 1 also exhausted that bound. | ||
| Each previous slice is partitioned into two without dropping fixtures or | ||
| changing compilation, snapshots, acceptance thresholds or smoke workers. |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,9 @@ | ||
| ### Fixed | ||
|
|
||
| - Apply declared GC witness settings during both compilation and execution of | ||
| the moving loop-poll arm, so seeded and protected witnesses select the matching | ||
| instrumented runtime. Validate the settings as literal GC assignments, keep | ||
| the compiled group separate from the safepoint control, and record the actual | ||
| compile and run settings. Exercise routing failure controls in the witness | ||
| workflow. Preserve the other arms, witness workloads, triage and movement | ||
| requirements. |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| ### Fixed | ||
|
|
||
| - Refresh the Windows classifications of `js_thread_global_materialize` and | ||
| `js_thread_global_publish` from the generated table produced by the documented | ||
| Windows archive classifier at the reviewed CI repair head. Materialization is | ||
| `AllocOnly` and publication is `Leaf` on this target. Preserve every other | ||
| table entry and all classifier rules and seeds, removing two conservative | ||
| drifts that fail the strict full-tier check. | ||
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| Register the immutable thread-global codegen IR suite in the CI source-to-suite map. This removes the unclassified-suite planner failure introduced when the suite landed and runs its three tests on codegen source changes, preserving mapped-suite timeouts and coverage independent of the named-suite cap. |
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
🔎 Supported by static analysis
🏁 Script executed:
Repository: PerryTS/perry
Length of output: 45616
🏁 Script executed:
Repository: PerryTS/perry
Length of output: 42526
Document the third Windows table change.
The target repair is committed, but the entry says that only two table entries changed. The committed Windows table also changes
js_arguments_object_map_indexfromReenterstoLeaf.Suggested changelog fix
📝 Committable suggestion
🤖 Prompt for AI Agents