Use `vec.extract_if(…)` for constant lifting #786

JonasWanke · 2023-11-02T15:28:28Z

Before: cargo run --release -- run packages/Examples/fibonacci.candy
  Time (mean ± σ):      2.273 s ±  0.051 s    [User: 1.966 s, System: 0.303 s]
  Range (min … max):    2.202 s …  2.353 s    10 runs

After: cargo run --release -- run packages/Examples/fibonacci.candy
  Time (mean ± σ):      2.140 s ±  0.031 s    [User: 1.842 s, System: 0.298 s]
  Range (min … max):    2.107 s …  2.193 s    10 runs

Checklist

Tests for the changes have been added (for bug fixes / features)
Docs have been added / updated (for bug fixes / features)

JonasWanke added 2 commits November 2, 2023 15:19

Run benchmarks on pushes to main sequentially

25193af

Use vec.extract_if(…) for constant lifting

2fc3b46

JonasWanke added the T: Performance Type: Performance Improvements label Nov 2, 2023

jwbot added the P: Compiler: Frontend Package: The compiler frontend label Nov 2, 2023

jwbot enabled auto-merge November 2, 2023 15:28

github-actions bot assigned JonasWanke Nov 2, 2023

Use run ID i/o null as concurrency key

d5f3bda

JonasWanke requested a review from MarcelGarus November 2, 2023 15:43

MarcelGarus approved these changes Nov 7, 2023

View reviewed changes

jwbot merged commit f79988f into main Nov 7, 2023
11 checks passed

jwbot deleted the use-extract-if branch November 7, 2023 16:35

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Use `vec.extract_if(…)` for constant lifting #786

Use `vec.extract_if(…)` for constant lifting #786

JonasWanke commented Nov 2, 2023 •

edited

Loading

Use vec.extract_if(…) for constant lifting #786

Use vec.extract_if(…) for constant lifting #786

Conversation

JonasWanke commented Nov 2, 2023 • edited Loading

Checklist

Use `vec.extract_if(…)` for constant lifting #786

Use `vec.extract_if(…)` for constant lifting #786

JonasWanke commented Nov 2, 2023 •

edited

Loading