Brummel
c9a84b33b3
Tasks 1-4 land fully review-green and are the cohesive shippable
subset (the per-thread embedding ABI, fully wired + proven):
- T1: FIXED-FIRST alloc-guard baseline pin (RED captured -> GREEN at T4).
- T2: runtime/rc.c gains ailang_ctx_t {alloc_count,free_count} +
ailang_ctx_new/_free + __thread __ail_tls_ctx; the two increment
sites become 'if (_ctx) _ctx->... else g_rc_...'. g_rc_* statics +
ailang_rc_stats_atexit + the constructor RETAINED VERBATIM as the
null-ctx (single-threaded executable) fallback. rc_accounting_tsan.c
+ driver: per-ctx 8x200000 tsan-clean (de-globalisation positive proof).
- T3: codegen Target::StaticLib forwarder gains a leading 'ptr %ctx'
+ one '@__ail_tls_ctx = external thread_local global ptr' decl +
TLS save/store/restore around the BYTE-UNCHANGED internal call;
_adapter/_clos + internal arg vector untouched (M1 decision held);
doc-comment provisionality narrowed to the value/record layout.
- T4: build_staticlib rejects --alloc != rc (RC-only swarm artefact).
Boss adjudication of the orchestrator's Task-5 BLOCKED (spec-defect,
correctly surfaced not hacked): swarm.c's -DSHARED_CTX negative
control is structurally impossible — examples/embed_backtest_step.ail
is a non-allocating scalar kernel that never writes a ctx field, and
__ail_tls_ctx is __thread, so a shared-ctx scalar swarm has no
shared-memory write to race on (the spec's OWN item-1 honesty point).
The de-globalisation teeth belong to the item-1 rc_accounting harness
(shared ctx genuinely races on ctx->alloc_count; orchestrator
independently confirmed tsan exit 66). Spec amended in lockstep
(Goal coherent-stop, must-fail-axis item 2, Testing item 3,
Acceptance) so item 3's negative control is item-1's by the
de-globalisation-proof / capability-demo split; plan Task 5
restructured (swarm.c per-ctx capability demo only; negative-control
standing test relocated into the item-1 harness driver) + Task 3's
plan-transcription error (@ail_backtest_step ->
@ail_embed_backtest_step_step) corrected. This is a Boss
consistency-repair of an internally-contradictory clause whose intent
is already correctly realised in the same spec — not a redesign; no
brainstorm bounce. Task-5 working files discarded (corrected plan
reproduces them; a structurally-RED test must not enter main).
INDEX.md + final journal land at iter completion (re-dispatch [5,9]).
56 lines
2.6 KiB
Rust
56 lines
2.6 KiB
Rust
//! Embedding-ABI M1 codegen (spec §4): `Target::StaticLib` lowering
|
|
//! emits NO `@main` trampoline, NO `MissingEntryMain`, and one
|
|
//! external `define i64 @backtest_step(...)` (the author-chosen C
|
|
//! symbol from `(export "backtest_step")`) forwarding to the
|
|
//! internal `@ail_embed_backtest_step_step` (module
|
|
//! `embed_backtest_step` == file stem, per Design-decision 6).
|
|
//! Executable-target lowering is byte-unchanged (the
|
|
//! `missing_entry_main_is_error` in-source pin still holds).
|
|
|
|
use ailang_core::Workspace;
|
|
use std::path::PathBuf;
|
|
|
|
fn load(file: &str) -> Workspace {
|
|
let path = PathBuf::from(env!("CARGO_MANIFEST_DIR"))
|
|
.parent().unwrap().parent().unwrap()
|
|
.join("examples").join(file);
|
|
ailang_surface::load_workspace(&path).unwrap()
|
|
}
|
|
|
|
#[test]
|
|
fn staticlib_emits_forwarder_no_main() {
|
|
let ws = load("embed_backtest_step.ail");
|
|
let ir = ailang_codegen::lower_workspace_staticlib_with_alloc(
|
|
&ws, ailang_codegen::AllocStrategy::Rc).unwrap();
|
|
assert!(!ir.contains("define i32 @main()"),
|
|
"staticlib IR must not emit the @main trampoline");
|
|
assert!(ir.contains("@backtest_step("),
|
|
"staticlib IR must emit the external @backtest_step entrypoint \
|
|
(the author-chosen C symbol — invariant under module rename)");
|
|
assert!(ir.contains("@ail_embed_backtest_step_step("),
|
|
"the forwarder must call the internal \
|
|
@ail_embed_backtest_step_step (module embed_backtest_step)");
|
|
|
|
// M2: forwarder takes a leading ptr %ctx and routes it through TLS
|
|
// around the UNCHANGED internal call.
|
|
assert!(ir.contains("@__ail_tls_ctx = external thread_local global ptr"),
|
|
"staticlib module must declare the external thread-local ctx slot;\nIR:\n{ir}");
|
|
assert!(ir.contains("define i64 @backtest_step(ptr %ctx, i64 %a0, i64 %a1)"),
|
|
"forwarder must take a leading ptr %ctx;\nIR:\n{ir}");
|
|
assert!(ir.contains("store ptr %ctx, ptr @__ail_tls_ctx"),
|
|
"forwarder must publish ctx into the TLS slot;\nIR:\n{ir}");
|
|
assert!(ir.contains("call i64 @ail_embed_backtest_step_step(i64 %a0, i64 %a1)"),
|
|
"internal call must stay byte-unchanged (no ctx arg);\nIR:\n{ir}");
|
|
}
|
|
|
|
#[test]
|
|
fn executable_target_still_emits_main() {
|
|
let ws = load("embed_backtest_step.ail");
|
|
// Default executable lowering is unaffected by the new field;
|
|
// backtest has no `main`, so exe-target lowering still rejects.
|
|
let err = ailang_codegen::lower_workspace_with_alloc(
|
|
&ws, ailang_codegen::AllocStrategy::Rc).unwrap_err();
|
|
assert!(format!("{err}").contains("has no `main` def"),
|
|
"exe target must still surface MissingEntryMain; got {err}");
|
|
}
|