Compare primitive values with RowFn

[connortsui20]

Depends on #10014.

Summary

• Tracking Issue: Epic: Row-oriented scalar functions #9128
• Replaces Compare primitive values with RowFn #9703 under the renamed branch. The original discussion remains there.

Moves primitive comparisons onto RowFn so decoding, constants, validity, and packed Boolean output use the shared row executor.

Changes

PrimitiveCompare dispatches primitive types and comparison operators through multiversioned RowVisitor::visit_bool, replacing the primitive lane kernels and their operand wrapper. Float comparisons retain Vortex's total ordering.

Depends on #10014 for execution-allocator ownership of Boolean output. The comparison tests added in #9948 are retained and now exercise the RowFn implementation.

Validation before the rebase: 168 focused comparison, mask, and RowFn tests passed on the combined stack, including comparison_uses_execution_allocator, along with cargo clippy -p vortex-array --all-targets --all-features -- -D warnings. Tests, formatting, and benchmarks were not rerun locally after the rebase.

[codspeed]

Merging this PR will regress 4 benchmarks

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

⚠️ 15 benchmarks spent significant time in system calls

System calls cannot be consistently instrumented, so they are not included in the measure, which understates the real cost. Please switch to the Walltime instrument to accurately measure system calls.

Measurement and system calls

• random_i128[0.01]
• chunked_dict_primitive_into_canonical[f64, (1000, 1000, 10)]
• take_varbin
• density_sweep_dense_runs[0.001]
• compress_fsst[(500, 16, 8)]
• decompress[u32, (1000, 256)]
• chunked_dict_primitive_canonical_into[u64, (1000, 1000, 10)]
• multipolygon_mixed_validity
• bench_compare_sliced_dict_primitive[(3333, 10000)]
• decompress[u16, (4000, 256)]
• …and 5 more

⚠️ Different runtime environments detected

Some benchmarks with significant performance changes were compared across different runtime environments,
which may affect the accuracy of the results.

Open the report in CodSpeed to investigate

⚡ 30 improved benchmarks
❌ 4 regressed benchmarks
✅ 2179 untouched benchmarks

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

	Mode	Benchmark	BASE	HEAD	Efficiency
❌	WallTime	threshold_avx2[8]	2 µs	2.4 µs	-18.01%
❌	WallTime	threshold_neon[8]	2.9 µs	3.5 µs	-16.8%
❌	WallTime	threshold_avx2[64]	3 µs	3.4 µs	-11.21%
❌	WallTime	threshold_neon[80]	4.7 µs	5.3 µs	-11.21%
⚡	WallTime	compare_f32_avx2	4.4 µs	2.6 µs	+67.05%
⚡	WallTime	compare_f32_avx512	3.4 µs	2.4 µs	+44.24%
⚡ ⚠️	Simulation	random_i128[0.01]	7.7 µs	5.5 µs	+38.51%
⚡	WallTime	mul_u64_nonnull_neon	39.6 µs	28.7 µs	+38.21%
⚡	WallTime	compare_float_neon	8.2 µs	6 µs	+36.62%
⚡	WallTime	compare_f32_neon	5.5 µs	4.2 µs	+31.49%
⚡	WallTime	compare_u64_avx2	3.7 µs	3 µs	+25.67%
⚡	WallTime	compare_float_avx2	4.9 µs	3.9 µs	+23.81%
⚡	WallTime	deferred_bool_neon[i64, PerRowPerRow]	7.2 µs	6.1 µs	+19.16%
⚡	WallTime	compare_int_constant_neon	4 µs	3.4 µs	+18.43%
⚡	WallTime	mul_i64_nonnull_neon	38.7 µs	32.6 µs	+18.43%
⚡	WallTime	multiply_shapes_neon[(32768, PerRowPerRow)]	38.6 µs	32.6 µs	+18.28%
⚡	WallTime	compare_u8_neon	2 µs	1.7 µs	+15.57%
⚡	WallTime	compare_u64_neon	4.5 µs	3.9 µs	+15.56%
⚡	WallTime	words_gather_dispatch_neon[1048576]	29.8 µs	25.8 µs	+15.31%
⚡	WallTime	compare_int_neon	4.5 µs	3.9 µs	+14.38%
...	...	...	...	...	...

ℹ️ Only the first 20 benchmarks are displayed. Go to the app to view all benchmarks.

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-compare-primitive (7a7559b) with develop (1a9ed68)}

[github-actions]

This PR has been marked as stale because it has been open for 14 days with no activity. Please comment or remove the stale label if you wish to keep it active, otherwise it will be closed in 7 days