Repository navigation
[AUTOMATED] Reuse immutable SLEIGH catalogs and SSA reachability - #204
Merged
Merged
Conversation
This was referenced Oct 15, 2026
…ation-performance
sjkim1127
marked this pull request as ready for review
October 19, 2026 10:17
sjkim1127
enabled auto-merge
October 19, 2026 10:17
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.
[AUTOMATED]
Scope
Mechanical removal of duplicate preparation work, with the existing producer contract retained:
No production dependency, CLI/schema, worker policy, recovery pass, type policy or timeout change. No control-flow snapshot borrowing is included: that experiment was excluded after an unresolved large-input slowdown.
Dependency
#203 has merged as
d471210842c1eb064ba216e5a6bc80d66a73f7ef. This PR is now retargeted to main and includes the required branch update. Its diff contains only the mechanical optimization and its evidence.Measured evidence
Frozen qualification corpus, revision
07dd95dbe6214b8a78fc1e4a55961ab515986489:The user paused other builds/benchmarks; no builds/tests from this task ran during the controlled timings. OS caches were not flushed and desktop scheduling remains a limitation. The evidence does not establish a corpus-wide speedup, Kuna parity, quality/ranking improvement or source execution parity. No corpus executable was run.
See
docs/validation/decompiler-preparation-performance.mdand its compact JSON for raw paired times, population, hashes, exclusions, commands and rejected-experiment details. Full raw outputs/profiles are retained locally underbenchmark/artifacts/performance/2026-10-14.Related datasets: qualification corpus, earlier raw corpus.
Validation
shared_token_operands_do_not_require_legacy_cursor_policy,test_walk_stack_backtrace_on_failure) occurred in an initial SLEIGH run. Subsequent full and isolated fresh-cache runs passed on both builds, and final remote CI passed. The initial cause is not established; counts and raw logs are documented.efe35ee6929ec16851f44f283c1ffef3ced70fb9: Linux/macOS tests, Linux native debugger tests, optional-feature checks, lint/security/owner gates and the real PE CLI smoke. It was dispatched explicitly because the stacked base is outside the main-only PR trigger.Current-main integration review (2026-10-19)
1d6acefd0b9d7e124197cb10ef6bb9c2d18ed78e. Its complete Git tree matches the locally validated integration tree exactly; branch updates introduced no production-content change.Ready for the bounded mechanical change after required checks pass. No further speed investigation is included in this review cycle.