RazrFalcon / tiny-skia

A tiny Skia subset ported to Rust
BSD 3-Clause "New" or "Revised" License
1.12k stars 69 forks source link

SIMD on AArch64 is now available on stable Rust #46

Closed Shnatsel closed 2 years ago

Shnatsel commented 2 years ago

The README states:

Skia also supports ARM NEON instructions, which are unavailable in a stable Rust at the moment. Therefore a fallback scalar implementation will be used instead on ARM and other non-x86 targets. So if you're targeting ARM, you better stick with Skia.

However, Rust now supports AArch64 SIMD intrinsics on stable starting with v1.59; see https://github.com/rust-lang/stdarch/pull/1266 for details.

Note that documentation still displays the intrinsics as unstable due to a rustdoc bug: https://github.com/rust-lang/stdarch/issues/1268

CryZe commented 2 years ago

Yeah I have an open PR for it: https://github.com/RazrFalcon/tiny-skia/pull/39

I'll try to pick it up again.

Shnatsel commented 2 years ago

Oh, and I see you've been working on adding it to safe_arch and bytemuck as well! Amazing! Thank you so much!

yisibl commented 2 years ago

@CryZe Thank you for your efforts. Looking forward to the completion of PR https://github.com/RazrFalcon/tiny-skia/pull/39.