Thanks to visit codestin.com
Credit goes to github.com

Skip to content

sha1+sha2: make aarch64 CPU feature detection OS-independent#276

Merged
tarcieri merged 1 commit into
masterfrom
sha1+sha2/remove-os-specific-aarch64-cpu-feature-detection
May 15, 2021
Merged

sha1+sha2: make aarch64 CPU feature detection OS-independent#276
tarcieri merged 1 commit into
masterfrom
sha1+sha2/remove-os-specific-aarch64-cpu-feature-detection

Conversation

@tarcieri
Copy link
Copy Markdown
Member

@tarcieri tarcieri commented May 15, 2021

cpufeatures v0.1.4 can build on any aarch64 target, not just Linux/macOS ones:

RustCrypto/utils#40

This PR bumps the minimum cpufeatures requirement to v0.1.4 and removes the OS-specifisms on aarch64 targets.

On platforms other than Linux/macOS, CPU features are always detected as false unless the corresponding target_features have been enabled via RUSTFLAGS. But in the future, we can add new platform support to cpufeatures without changing all of its downstream consumers.

@tarcieri tarcieri changed the title sha1+sha2: remove OS-specific aarch64 CPU feature detection sha1+sha2: make aarch64 CPU feature detection OS-independent May 15, 2021
`cpufeatures` v0.1.4 can build on any `aarch64` target, not just
Linux/macOS ones:

RustCrypto/utils#40

This PR bumps the minimum `cpufeatures` requirement to v0.1.4 and
removes the OS-specifisms on `aarch64` targets.
@tarcieri tarcieri force-pushed the sha1+sha2/remove-os-specific-aarch64-cpu-feature-detection branch from b82d1c8 to 656087c Compare May 15, 2021 19:12
@tarcieri tarcieri merged commit 1de4114 into master May 15, 2021
@tarcieri tarcieri deleted the sha1+sha2/remove-os-specific-aarch64-cpu-feature-detection branch May 15, 2021 19:29
@dignifiedquire
Copy link
Copy Markdown
Member

nice

@tarcieri tarcieri mentioned this pull request Jul 18, 2021
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.

2 participants