Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

fn decode_b: Performance improvements #1320

Merged
merged 2 commits into from
Jul 16, 2024
Merged

fn decode_b: Performance improvements #1320

merged 2 commits into from
Jul 16, 2024

Commits on Jul 16, 2024

  1. Inline backup2x8 

    This function is inlined in C and we see a small
performance improvement inlining it in Rust (~0.5%
on a Ryzen 7700X for Chimera 8-bit).
    @rinon
    rinon committed Jul 16, 2024
    Configuration menu
    Copy the full SHA
    ba5f766 View commit details
    Browse the repository at this point in the history

  2. fn decode_b: Make local variable for &f.a[t.a] 

    The `f.a[t.a]` block context reference is constant
throughout this function, but it appears that the
function is too complex for the optimizer to not
recompute this reference. Making it a local
improves performance measurably (~1% on a Ryzen
7700X for 8-bit Chimera).
    @rinon
    rinon committed Jul 16, 2024
    Configuration menu
    Copy the full SHA
    f63c2ab View commit details
    Browse the repository at this point in the history