Index values in variable-arity sequence tables - #975
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## codex/inline-containers-01-sequence-table #975 +/- ##
=============================================================================
+ Coverage 87.56% 87.64% +0.08%
=============================================================================
Files 97 97
Lines 36183 36491 +308
=============================================================================
+ Hits 31683 31982 +299
- Misses 4500 4509 +9 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Merging this PR will degrade performance by 7.25%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Simulation | tests[eggcc-2mm] |
7.9 s | 8.7 s | -9.42% |
| ❌ | Simulation | rust_rule_match_overhead[rule_run_1] |
13.8 ms | 14.6 ms | -5.03% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing codex/inline-containers-02-sequence-indexes (c6a903a) with codex/inline-containers-01-sequence-table (1b22eca)
Footnotes
-
227 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩
Summary
Stack
Draft, stacked on #974. This is PR 2 of 12 in the sequence-backed container stack.
Validation
Validated on the complete 12-commit stack:
cargo nextest run --workspace: 1,362 tests passedcargo test --workspace --doccargo clippy --workspace --all-targets -- -D warningscargo fmt --all -- --check