22af40413d
Stream A (atomics) foundation. Net-new atomic load/store codegen path, wired end-to-end except LLVM emission, which deliberately bails loudly so the example locks to a clean diagnostic (A.0b turns it green — cadence: no commit both adds a test and makes it pass). - library/modules/std/atomic.sx: Ordering enum, Atomic($T) transparent wrapper (init/load/store, seq_cst-only for now), atomic_load/atomic_store #builtin intrinsics. Opt-in import, NOT in the universal std facade (Ordering in the prelude grows every program's type table + churns 37 .ir snapshots). - IR: atomic_load/atomic_store ops + AtomicOrdering (all 5) + structs (inst.zig); print arms; comptime_vm arms reuse load/store (single-thread correct); recognizer tryLowerAtomicIntrinsic (const-ordering + scalar-size guards, both loud); emit dispatch -> emitAtomicLoad/Store bail via comptime_failed. - examples/1700-atomics-load-store.sx locked to the bail diagnostic. Full ordering surface (a.load(.acquire)) blocked on comptime-constant ordering propagation (comptime enum value params) — A.0.5, migrated not legacy.
67 lines
4.3 KiB
Markdown
67 lines
4.3 KiB
Markdown
# CHECKPOINT-ATOMICS — Stream A (atomics lowering)
|
|
|
|
Companion to [PLAN-ATOMICS.md](PLAN-ATOMICS.md). Update after every step (one step at a
|
|
time, per the cadence rule). New corpus category: `17xx`.
|
|
|
|
## Last completed step
|
|
**A.0a (lock commit) — DONE.** Full atomic load/store plumbing landed with LLVM emission
|
|
deliberately bailing loudly; `examples/1700-atomics-load-store.sx` locked to the bail
|
|
diagnostic (exit 1). Suite green (710 examples, 0 failed; 476 units).
|
|
- `library/modules/std/atomic.sx`: `Ordering` enum, `Atomic($T)` struct (`init`/`load`/
|
|
`store`, **seq_cst-only** — see capability gap below), `atomic_load`/`atomic_store`
|
|
`#builtin` decls. **Opt-in import**, NOT in the universal `std.sx` facade (mirrors
|
|
`trace`) — putting `Ordering` in the prelude grew every program's type table 378→380 and
|
|
churned 37 `.ir` snapshots; reverted.
|
|
- IR ops `atomic_load`/`atomic_store` + `AtomicOrdering` (all 5) + structs (inst.zig);
|
|
print arms (print.zig); comptime_vm arms reuse load/store (single-thread correct);
|
|
recognizer `tryLowerAtomicIntrinsic` (call.zig) — const-ordering-literal guard +
|
|
scalar-size guard, both loud; emit dispatch arms (emit_llvm.zig) → `emitAtomicLoad`/
|
|
`emitAtomicStore` (ops.zig) currently BAIL via `comptime_failed`.
|
|
|
|
## Current state
|
|
- A.0a committed; suite green.
|
|
- The recognizer + IR + emit already handle ALL FIVE orderings; only the surface bakes a
|
|
`.seq_cst` literal (the methods can't yet forward a runtime/comptime ordering — gap below).
|
|
- emit bodies are the ONLY placeholder; A.0b swaps them for real builders.
|
|
|
|
## Next step
|
|
**A.0b** — replace the `emitAtomicLoad`/`emitAtomicStore` bail with real
|
|
`LLVMBuildLoad2`+`SetOrdering`+`SetAlignment` / `LLVMBuildStore`+`SetOrdering`+
|
|
`SetAlignment` (explicit sx-tag→LLVM ordering switch); regen 1700 → green (7/42/43) + host
|
|
`.ir`; add `emit_llvm.test.zig` unit. Then adversarial review, then the comptime-enum worker
|
|
+ A.0.5 migration to the full ordering surface.
|
|
|
|
## Known issues / capability gaps
|
|
- **Comptime-constant ordering propagation MISSING (blocks the full surface).** A runtime
|
|
`Ordering` method param can't reach LLVM (orderings are instruction attributes, not
|
|
operands), and comptime enum value params don't exist (`$o: Ordering` → `o` unresolved in
|
|
body; `resolveValueParamArg` folds integers only). So A.0 ships seq_cst-only; A.0.5 closes
|
|
the gap (worker: implement comptime enum value params) and MIGRATES the methods — NO
|
|
legacy left by stream end.
|
|
- **Latent (observed, not yet filed):** calling an *unrecognized* bodiless `#builtin`
|
|
silently returns 0 / no-ops with exit 0 (that's how 1700 behaved before recognition
|
|
landed) — a silent-fallback footgun in the generic builtin-call path, independent of
|
|
atomics. Flag to user; candidate `issues/` entry.
|
|
|
|
## Decisions (Stream A specifics; surface locked in design §4.6)
|
|
- `Atomic($T)` = pure-sx transparent 1-field struct (NO new IR type); ops = `#builtin`
|
|
intrinsics emitted as new IR ops. Minimal compiler surface.
|
|
- Ordering is compile-time-only (const enum literal), baked into the op as a Zig enum;
|
|
non-literal = loud diagnostic. sx tag → LLVM ordering via explicit switch (LLVM enum is
|
|
non-contiguous: 2/4/5/6/7).
|
|
- Atomic load/store REQUIRE explicit alignment (`LLVMSetAlignment`) — verifier mandate.
|
|
- Comptime VM treats atomics as ordinary load/store (single-thread ⇒ correct), not a bail.
|
|
- **Snapshot scope corrected:** `.ir` (LLVM IR) is arch-invariant for atomics → ONE host
|
|
`.ir` per op, not arch-gated x86/aarch64 pairs (they'd be byte-identical). Asm-level arch
|
|
divergence + weak-memory semantics are OUT of corpus scope (stress harness, Stream C).
|
|
|
|
## Log
|
|
- **carve** — wrote PLAN-ATOMICS.md + CHECKPOINT-ATOMICS.md; grounded the intrinsic path,
|
|
switch sites, LLVM-C API (no `LLVMBuildAtomicLoad`; use `LLVMBuildLoad2`+`SetOrdering`+
|
|
`SetAlignment`), and corrected the arch-`.ir` misconception (`sx ir` emits arch-invariant
|
|
LLVM IR). Stream ready; A.0a is the first implementation step.
|
|
- **A.0a** — landed lib (atomic.sx, opt-in import) + IR ops (atomic_load/atomic_store +
|
|
AtomicOrdering) + recognizer + print/vm arms + emit BAIL; locked `examples/1700` to the
|
|
bail diagnostic. Reverted a universal-facade wiring that churned 37 `.ir` snapshots
|
|
(Ordering would bloat every program's type table). Suite green (710/0).
|