Skip to content

Commit

Permalink
[OPT] Don't force return value of call to stack
Browse files Browse the repository at this point in the history
Benchmark #1: ./simple_raytracer_before
  Time (mean ± σ):     14.420 s ±  0.568 s    [User: 14.376 s, System: 0.026 s]
  Range (min … max):   13.730 s … 15.170 s    10 runs

Benchmark #2: simple_raytracer_after
  Time (mean ± σ):     13.679 s ±  0.576 s    [User: 13.628 s, System: 0.020 s]
  Range (min … max):   12.761 s … 14.552 s    10 runs

Summary
  './simple_raytracer_after' ran
    1.05 ± 0.06 times faster than './simple_raytracer_before'
  • Loading branch information
bjorn3 committed Dec 20, 2019
1 parent a528e37 commit a125b62
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions src/analyze.rs
Original file line number Diff line number Diff line change
Expand Up @@ -30,14 +30,6 @@ pub fn analyze(fx: &FunctionCx<'_, '_, impl Backend>) -> IndexVec<Local, SsaKind
_ => {}
}
}

match &bb.terminator().kind {
TerminatorKind::Call {
destination: Some((place, _)),
..
} => analyze_non_ssa_place(&mut flag_map, place),
_ => {}
}
}

flag_map
Expand Down

0 comments on commit a125b62

Please sign in to comment.