Skip to content

feat: extend IGLA RACE cross-target to systolic PE + gen-backend findings - #1881

Merged
gHashTag merged 1 commit into
masterfrom
feat/verify-igla-systolic
Aug 7, 2026
Merged

feat: extend IGLA RACE cross-target to systolic PE + gen-backend findings#1881
gHashTag merged 1 commit into
masterfrom
feat/verify-igla-systolic

Conversation

@gHashTag

@gHashTag gHashTag commented Aug 7, 2026

Copy link
Copy Markdown
Owner

Extends the IGLA RACE bridge up the datapath. verify_igla_race.py now also cross-checks systolic_ternary_pe (psum_out = psum_in + ternary_mul(a,w), i16 accumulator) across C + Rust + reference over 800 vectors incl. edges (a=-128, invalid codes, i16 psum saturation): C == Rust == reference bit-exact. ternary_mac and the systolic PE are both multi-target-verified.

Findings: (1) #1773 concrete in C/Rustsystolic_ternary uses + calls ternary_mul but the import is NOT emitted into its gen-c/gen-rust output (dangling reference); the primitive is supplied from ternary_mac's output. (2) gen-rust emits psum_in(i16) + prod(i8) with no cast — Rust rejects the mixed-width add (C promotes); worked around with a documented as i16. Wired the systolic spec into the gate trigger. Refs #1764

…ings

verify_igla_race.py now also cross-checks systolic_ternary_pe (psum_out = psum_in +
ternary_mul(a,w), i16 accumulator) across C + Rust + reference over 800 vectors incl.
edges: C == Rust == reference bit-exact. ternary_mac and the systolic PE are both
multi-target-verified.

Finding (#1773 concrete in C/Rust): systolic_ternary uses/calls ternary_mul but the
import is NOT emitted into its gen-c/gen-rust output -- a dangling reference; the
primitive is supplied from ternary_mac's output. Finding (gen-rust): emits
psum_in(i16) + prod(i8) with no cast -- Rust rejects the mixed-width add (C promotes);
worked around with a documented  to cross-check the arithmetic. Wired the
systolic spec into the gate trigger.

Refs #1764

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

📓 NotebookLM Notebook linked to this PR

This notebook contains session context, decisions, and artifacts for this work.

@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-08-07 15:20:58 UTC

Summary

Status Count
Total Open PRs 13
PRs with Failing Checks 1
PRs with All Checks Green 12
READY 5
FAILING 1
PENDING 0

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=4c1aaad5309a != manifest seal=87e5cbd3ad94.
    The committed NMSE numbers were certified against an older compiler.rs.
    Run scripts/reseal-check.sh locally for the two-step reseal command (advisory; not a merge gate).

@gHashTag
gHashTag merged commit eb16623 into master Aug 7, 2026
18 checks passed
@gHashTag
gHashTag deleted the feat/verify-igla-systolic branch August 7, 2026 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant