Add tensor row inputs for RowFn - #9776
Conversation
Signed-off-by: Connor Tsui <connor.tsui20@gmail.com>
Merging this PR will regress 1 benchmark
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Simulation | random_i8[0.5] |
67.7 µs | 91.4 µs | -25.88% |
| ⚡ | WallTime | arrow_checked_add_u32_neon[16384] |
20.4 µs | 12.8 µs | +59.86% |
| ⚡ | Simulation | random_i16[0.8] |
93.6 µs | 75.3 µs | +24.25% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing ct/row-fn-tensor-rows (5eba5b2) with ct/tensor-normalization-semantics (fc92123)
Footnotes
-
218 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
Adds the tensor row types used by RowFn scalar functions without migrating a function in the same layer.
Changes
TensorRowdecodes ordinary tensor columns, literal constants, and extension-backed constants into flat typed rows. It is dense-safe because tensor float kernels can consume the stored payload of null rows while RowFn owns output validity.