ffi issue-0048: bare $args slice loses .len across call — xfail lock-in
Bare `$args` evaluated inside a pack-fn body has the right `.len` / per-element types inline, but the moment the same slice is passed as an argument to another function, the callee silently reads length 0 and every element comes back as undef. Cause (per issue file): `lazyLowerFunction` saves/restores builder state but not `pack_arg_nodes` / `pack_param_count` / `pack_arg_types` / `inline_return_target`. When a regular fn like `describe(args: []Any)` is lazily lowered from inside a pack-fn mono, the outer pack maps are still active; `lowerFieldAccess`'s `<pack_name>.len` intercept fires on `describe`'s same-named param and bakes the outer mono's arity as a constant into describe's IR. Every subsequent shape's call to describe returns that constant. `examples/173-pack-bare-args-cross-call.sx` exercises four shapes (0, 1, 3, 5 elements) through the same `describe(args: []Any)` walker. The expected output holds the per-position type names (`[s64]`, `[s64, string, bool]`, etc); today's diff fails — the walker reads `args.len = 0` for every shape and returns `[]`. The next commit fixes `lazyLowerFunction`.
This commit is contained in:
49
examples/173-pack-bare-args-cross-call.sx
Normal file
49
examples/173-pack-bare-args-cross-call.sx
Normal file
@@ -0,0 +1,49 @@
|
||||
// Regression: bare `$args` slice survives crossing a function-call
|
||||
// boundary — both `.len` AND per-element values come through.
|
||||
//
|
||||
// Before the fix landed in `lazyLowerFunction`, the callee's
|
||||
// `args.len` got constant-folded to the outer pack-fn mono's
|
||||
// arity. `walk(args: []Any) { return args.len; }` lazily lowered
|
||||
// inside `probe(..$args)`'s first mono inherited
|
||||
// `pack_param_count["args"] = N` from the pack — the
|
||||
// `<pack_name>.len` intercept in `lowerFieldAccess` then baked
|
||||
// `ret i64 N` into walk's IR. Every subsequent shape's call to
|
||||
// walk returned the same constant, regardless of the actual slice
|
||||
// it received.
|
||||
//
|
||||
// `describe(args)` walks element-by-element so a silent
|
||||
// truncation surfaces as a missing tail (or a different type at
|
||||
// some position) — not just the wrong length.
|
||||
//
|
||||
// Walking under `#run` is intentional: the bare-`$args` slice
|
||||
// carries `const_type` elements that only the interp materialises;
|
||||
// LLVM emission leaves the per-element slots as undef (4A.bare
|
||||
// semantics — bare-pack is comptime-only).
|
||||
|
||||
#import "modules/std.sx";
|
||||
|
||||
describe :: (args: []Any) -> string {
|
||||
s := "[";
|
||||
i : s64 = 0;
|
||||
while i < args.len {
|
||||
if i > 0 { s = concat(s, ", "); }
|
||||
s = concat(s, type_name(args[i]));
|
||||
i = i + 1;
|
||||
}
|
||||
return concat(s, "]");
|
||||
}
|
||||
|
||||
probe :: (..$args) -> string {
|
||||
list := $args;
|
||||
return describe(list);
|
||||
}
|
||||
|
||||
run_all :: () {
|
||||
print("0: {}\n", probe());
|
||||
print("1: {}\n", probe(1));
|
||||
print("3: {}\n", probe(1, "x", true));
|
||||
print("5: {}\n", probe(1, 2.0, "x", true, 99));
|
||||
}
|
||||
#run run_all();
|
||||
|
||||
main :: () { print("rt\n"); }
|
||||
Reference in New Issue
Block a user