shish / rosettaboy

A gameboy emulator in several different languages
MIT License
469 stars 22 forks source link

[nim] +release-lto #136

Closed inv2004 closed 1 year ago

inv2004 commented 1 year ago

release+lto could be interesting too because "danger" gives some boost, but also disables some checks - it is not default choose in many cases, but release+lto is always ok

shish commented 1 year ago

So running this a whole bunch of times, for several minutes to try and even out any noise, just LTO is consistently coming up ~1% faster than LTO + danger + other flags :O

inv2004 commented 1 year ago

Please do not delete danger/speed, because it cannot be slower. Probably some problems with flags or smth like that - necessary to check. Danger should be faster - interesting to investigate

shish commented 1 year ago

Seems weird but keeping a danger build doesn't seem harmful, so sure we can have both :) I'll leave it to somebody who understands nim compiler flags to investigate though ^^;