tweag / nickel

Better configuration for less
https://nickel-lang.org/
MIT License
2.43k stars 93 forks source link

[RFC007] Conversion from new AST to mainline AST #2080

Closed yannham closed 3 weeks ago

yannham commented 4 weeks ago

Depends on #2079.

Add a the inverse transformation to bytecode::compat (to go from the new AST to the mainline AST), following our new plan of replacing the left part of the pipeline (parsing, typechecking ,etc.) step by step to use the new AST including in mainline Nickel, and just translate to the old one for the passes that aren't yet upgraded, provided that we measure that the AST conversion is negligible.

github-actions[bot] commented 4 weeks ago

🐰 Bencher Report

Branch2080/merge
Testbedubuntu-latest

⚠️ WARNING: The following Measure does not have a Threshold. Without a Threshold, no Alerts will ever be generated!

Click here to create a new Threshold
For more information, see the Threshold documentation.
To only post results if a Threshold exists, set the --ci-only-thresholds CLI flag.

Click to view all benchmark results
BenchmarkLatencynanoseconds (ns)
fibonacci 10📈 view plot
⚠️ NO THRESHOLD
485,500.00
foldl arrays 50📈 view plot
⚠️ NO THRESHOLD
1,718,900.00
foldl arrays 500📈 view plot
⚠️ NO THRESHOLD
6,597,000.00
foldr strings 50📈 view plot
⚠️ NO THRESHOLD
7,019,200.00
foldr strings 500📈 view plot
⚠️ NO THRESHOLD
61,708,000.00
generate normal 250📈 view plot
⚠️ NO THRESHOLD
46,787,000.00
generate normal 50📈 view plot
⚠️ NO THRESHOLD
2,022,100.00
generate normal unchecked 1000📈 view plot
⚠️ NO THRESHOLD
3,233,900.00
generate normal unchecked 200📈 view plot
⚠️ NO THRESHOLD
748,460.00
pidigits 100📈 view plot
⚠️ NO THRESHOLD
3,136,600.00
pipe normal 20📈 view plot
⚠️ NO THRESHOLD
1,500,600.00
pipe normal 200📈 view plot
⚠️ NO THRESHOLD
9,997,100.00
product 30📈 view plot
⚠️ NO THRESHOLD
818,750.00
scalar 10📈 view plot
⚠️ NO THRESHOLD
1,510,200.00
sum 30📈 view plot
⚠️ NO THRESHOLD
811,910.00
🐰 View full continuous benchmarking report in Bencher
dpulls[bot] commented 3 weeks ago

:tada: All dependencies have been resolved !