search

TeXitoi / benchmarksgame-rs

The Computer Language Benchmarks Game: Rust implementations
Other
70 stars 19 forks source link

mandelbrot: inline and remove LTO from Makefile #52

Closed cristicbz closed 7 years ago

cristicbz commented 7 years ago

Fixes #50, I think: removing LTO somehow changed LLVM's inlining heuristics and advance didn't get inlined anymore. Adding the #[inline] annotation brings down the runtime to the same value as LTO.

TeXitoi commented 7 years ago

Will you take care of the submission?

cristicbz commented 7 years ago

Yup, ok, will do since I did the previous one, thanks!

TeXitoi commented 7 years ago

Thanks! All these improvement are really exciting!

cristicbz commented 7 years ago

https://alioth.debian.org/tracker/index.php?func=detail&aid=315635&group_id=100815&atid=413122

cristicbz commented 7 years ago

@BurntSushi So the benchmarks game maintainer said you're the person to talk about removing LTO, do you know what the status is on this: https://alioth.debian.org/tracker/index.php?func=detail&aid=315635&group_id=100815&atid=413122

The linked bug is two years old so I imagine it's all good?