yggdrasil/ring
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

AES-GCM: Clarify that we need AVX (AVX1), not AVX2.

Browse Source 
The function was named `is_avx2()` but the test is for AVX1. Fix the name.
This commit is contained in:
Brian Smith 2023-11-26 12:57:49 -08:00
parent 6c29bf61cd
commit b3d0b6f4b3
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/aead/aes_gcm.rs
View File

@ -84,7 +84,7 @@ fn aes_gcm_seal(key: &aead::KeyInner, nonce: Nonce, aad: Aad<&[u8]>, in_out: &mu
#[cfg(target_arch = "x86_64")]
let in_out = {
if !aes_key.is_aes_hw() || !auth.is_avx2() {
if !aes_key.is_aes_hw() || !auth.is_avx() {
in_out
} else {
use crate::c;
@ -164,7 +164,7 @@ fn aes_gcm_open(
#[cfg(target_arch = "x86_64")]
let in_out = {
if !aes_key.is_aes_hw() || !auth.is_avx2() {
if !aes_key.is_aes_hw() || !auth.is_avx() {
in_out
} else {
use crate::c;

2
src/aead/gcm.rs
View File

@ -244,7 +244,7 @@ impl Context {
}
#[cfg(target_arch = "x86_64")]
pub(super) fn is_avx2(&self) -> bool {
pub(super) fn is_avx(&self) -> bool {
match detect_implementation(self.cpu_features) {
Implementation::CLMUL => has_avx_movbe(self.cpu_features),
_ => false,