Skip to content

Commit 91c23d1

Browse files
committed
Reject discarded binary branch fusion
1 parent 13e5d22 commit 91c23d1

2 files changed

Lines changed: 47 additions & 6 deletions

File tree

tasks/README.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -109,11 +109,11 @@ Campaign working rules:
109109
local hits through frame-verified direct variants, with all 2,040 runtime
110110
tests passing. HashMap improved only to 0.9841x candidate/base, missing the
111111
frozen 0.97 target, so all runtime changes were reverted.
112-
- `T028-discarded-binary-branch-superinstruction.md`**planned.** Exact
113-
dynamic opcode evidence finds a two-edge discarded binary condition capable
114-
of removing 9.73% of HashMap dispatch, 5.17% of CDJS dispatch, and 2.73% of
115-
public-field Raytrace dispatch without changing source bytecode or loop-plan
116-
offsets. Its rank-one one-attempt plan is frozen before implementation.
112+
- `T028-discarded-binary-branch-superinstruction.md`**rejected.** The
113+
prototype covered 99.9999% of the frozen HashMap sequence and removed 9.73%
114+
of its generic dispatch with all 2,039 prototype runtime tests passing, but
115+
improved HashMap only to 0.9837x candidate/base against the frozen 0.97
116+
target. The one-attempt runtime changes were reverted.
117117
- `T022-performance-priority-controller.md` — evidence-bound selection and
118118
stop mechanism for T018. It derives the next profiling queue from current
119119
artifacts, freezes a unit's targets before timing, and records explicit

tasks/T028-discarded-binary-branch-superinstruction.md

Lines changed: 42 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# T028: Discarded binary-branch superinstruction
22

3-
## Status: planned from exact evidence; implementation not started
3+
## Status: rejected after the frozen one-attempt gate
44

55
This T018 leaf unit is frozen in
66
`performance-units/discarded-binary-branch-superinstruction.json` against the
@@ -110,3 +110,44 @@ Forbidden within this unit:
110110
only if the target passes.
111111
6. Record `retained`, `rejected`, or `inconclusive` without changing the cases,
112112
thresholds, or one-attempt budget after timing.
113+
114+
## Result
115+
116+
The one-attempt prototype appended the derived opcode while preserving the
117+
exact 96-byte `Op` layout. Lowering required the complete flow analysis, the
118+
exact two-`Pop` successor shape, checked targets, and a linear source range;
119+
the existing local-local comparison fusion retained priority. Focused tests
120+
covered both numeric outcomes, strings, observable left-to-right object
121+
coercion, thrown coercion, surrounding stack values, missing edge Pops,
122+
malformed targets, unproven ranges, and incomplete analysis. All 2,039
123+
prototype qjs-runtime tests passed, the 13 perf-counter diagnostics passed,
124+
and runtime clippy with `-D warnings` passed.
125+
126+
The mechanism gate passed. The diagnostic candidate executable SHA-256 was
127+
`e5c1626914fc6138542e7b837b2828f5c0fe8eff0a28833310d30188e586d220`.
128+
It dispatched the fused operation 5,122,203 times, covering 99.9999% of the
129+
5,122,209 frozen HashMap sequences, and reduced `executed_ops` from
130+
105,261,606 to 95,017,200: 10,244,406 fewer dispatches, or 9.73%. The counter
131+
receipt SHA-256 is
132+
`16a7ba8d095622c06fccf3b608e142d7d6f9ff28f0b6922a63f7efa395cd4d0c`.
133+
134+
The frozen payoff gate nevertheless failed. The exact base and standard
135+
prototype executable SHA-256 values were
136+
`f6bd01fd4fedabfa16ec133f937b6f3dc2476e31e3fdd680b037145c3d8aebe1`
137+
and
138+
`ab97ef4892c9af6c3fc9b1b790e914602c1d07bf63fccc9b08b2aa3953d4f7ed`.
139+
After one warmup per role, eleven strictly alternating HashMap pairs measured
140+
median candidate/base **0.983685x**, missing the required `<= 0.97x`. Median
141+
base and candidate times were 1.6493s and 1.6235s; all pair ratios were
142+
0.974149-0.987993. The A/B receipt SHA-256 is
143+
`431ac268b16478c7b16e3fca7d0adc171f19c1bfd72fe37f59c1667a6df5047a`.
144+
145+
Because the sole payoff target failed, the frozen controls and complete
146+
promotion runs were unwarranted. The opcode, lowering, dispatch, diagnostics,
147+
and focused tests were reverted; the checked-in runtime is source-identical to
148+
the frozen base. The rejected prototype diff SHA-256 is
149+
`08a6601734a3c7127e5ea5f892d0f173c597d1742ef4a1d18966d743dc173422`.
150+
Do not retry this discarded binary-branch fusion or merely move its handler;
151+
a successor dispatch proposal needs new exact evidence and a structurally
152+
different mechanism capable of clearing the remaining roughly 1.4 percentage
153+
points to the frozen threshold.

0 commit comments

Comments
 (0)