Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Add inline annotations #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add inline annotations #8

wants to merge 1 commit into from

Commits on Sep 26, 2022

  1. Add inline annotations 

    Without enabling link time optimization (LTO) there will never be any
inlining across crate boundaries. These methods will be mostly called
from other crates, they are small and used in hot loops.

On a branch of rust-png-reader that was made to use Circular this change
reduced the runtime of one benchmark test from 3 minutes 30 seconds to 1
minute 40 seconds (so over 50%).

(The benchmark is running the example/stats binary on a large (50GB) PGN
file downloaded from the lichess DB)
    Joeri Samson committed Sep 26, 2022
    Configuration menu
    Copy the full SHA
    1689327 View commit details
    Browse the repository at this point in the history