Skip to content

cargo: nuke 'simd-accel' from orbit#2749

Merged
BurntSushi merged 1 commit intomasterfrom
ag/nuke-simd-accel
Mar 7, 2024
Merged

cargo: nuke 'simd-accel' from orbit#2749
BurntSushi merged 1 commit intomasterfrom
ag/nuke-simd-accel

Conversation

@BurntSushi
Copy link
Owner

This feature causes nothing but problems and is frequently broken. The
only optimization it was enabling were SIMD optimizations for
transcoding. In particular, for UTF-16 transcoding. This is performed by
the encoding_rs crate,
which specifically uses unstable portable SIMD APIs instead of the
stable non-portable SIMD APIs.

SIMD optimizations that apply to search have long been making use of
stable APIs, and are automatically enabled when your target supports
them. This is, IMO, the correct user experience and one that
encoding_rs refuses to support. I'm done dealing with it, so
transcoding will only use scalar code until the SIMD optimizations in
encoding_rs work on stable. (This doesn't mean that encoding_rs has
to change. This could also be fixed by stabilizing std::simd.)

Fixes #2748

This feature causes nothing but problems and is frequently broken. The
only optimization it was enabling were SIMD optimizations for
transcoding. In particular, for UTF-16 transcoding. This is performed by
the [`encoding_rs`](https://github.com/hsivonen/encoding_rs) crate,
which specifically uses unstable portable SIMD APIs instead of the
stable non-portable SIMD APIs.

SIMD optimizations that apply to search have long been making use of
stable APIs, and are automatically enabled when your target supports
them. This is, IMO, the correct user experience and one that
`encoding_rs` refuses to support. I'm done dealing with it, so
transcoding will only use scalar code until the SIMD optimizations in
`encoding_rs` work on stable. (This doesn't mean that `encoding_rs` has
to change. This could also be fixed by stabilizing `std::simd`.)

Fixes #2748
@BurntSushi BurntSushi merged commit e9abbc1 into master Mar 7, 2024
@BurntSushi BurntSushi deleted the ag/nuke-simd-accel branch March 7, 2024 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Since nightly-2024-02-06 , could not compile with --features 'simd-accel'

1 participant