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

Add vectorization sample. #1067

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Add vectorization sample. #1067

wants to merge 4 commits into from

Commits on Aug 27, 2024

  1. Upgrade dependencies.

    @DanAlbert
    DanAlbert committed Aug 27, 2024
    Configuration menu
    Copy the full SHA
    cefa399 View commit details
    Browse the repository at this point in the history

  2. Upgrade gradle and AGP.

    @DanAlbert
    DanAlbert committed Aug 27, 2024
    Configuration menu
    Copy the full SHA
    ab63232 View commit details
    Browse the repository at this point in the history

  3. Fix .clang-format to not preserve bad format. 

    Deriving pointer alignment essentially makes this do nothing, since for
whatever reason (fighting with Studio maybe?), for new files it's
deriving the wrong style.
    @DanAlbert
    DanAlbert committed Aug 27, 2024
    Configuration menu
    Copy the full SHA
    5c4c625 View commit details
    Browse the repository at this point in the history

  4. Add vectorization sample. 

    This adds a new generic vectorization sample to replace hello-neon. Most
importantly, it covers non-Neon options for SIMD. One of the useful
things it shows, for example, is that there's actually no reason to
write SIMD code the way that hello-neon does any more.

This also solves a much simpler problem (small matrix multiplication),
which makes it easier to see how to deal with the SIMD features rather
than figuring out what a FIR filter is.

Finally, this sample benchmarks each of the implementations so it's
obvious what is and isn't worth doing. I was sort of surprised that
auto-vectorization didn't do better, and was pleased to learn that
there's no reason at all to write Neon intrinsics.

I'll delete hello-neon after this merges and I've fixed up the doc
links.

android#1011
    @DanAlbert
    DanAlbert committed Aug 27, 2024
    Configuration menu
    Copy the full SHA
    e90f344 View commit details
    Browse the repository at this point in the history