| commit | f913458a1666fc8d5465ab3af8a086f19e67b7d1 | [log] [tgz] |
|---|---|---|
| author | Eric Biggers <ebiggers@google.com> | Tue Dec 10 22:02:40 2024 +0000 |
| committer | Boringssl LUCI CQ <boringssl-scoped@luci-project-accounts.iam.gserviceaccount.com> | Tue Dec 10 22:28:59 2024 +0000 |
| tree | f15fd4f17c14d6a63f1907f9d68e3a1d3cb87e74 | |
| parent | 57f525e425a95e7c89fa2df926fa93273bd47311 [diff] |
Add missing $ to immediate in aes-gcm-avx10-x86_64.pl Found by code review. It worked anyway because the line is only included in the Windows NASM file. But omitting the $ is inconsistent. No change to the generated asm files. Change-Id: I12aa43fd9d71d7a71bbfdf2a47de7f23dd493cfa Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/74107 Commit-Queue: David Benjamin <davidben@google.com> Reviewed-by: David Benjamin <davidben@google.com>
diff --git a/crypto/fipsmodule/modes/asm/aes-gcm-avx10-x86_64.pl b/crypto/fipsmodule/modes/asm/aes-gcm-avx10-x86_64.pl index b65dee9..9ff7367 100644 --- a/crypto/fipsmodule/modes/asm/aes-gcm-avx10-x86_64.pl +++ b/crypto/fipsmodule/modes/asm/aes-gcm-avx10-x86_64.pl
@@ -169,7 +169,7 @@ my $pos = 16 * $i; $code .= "movdqa $pos(%rsp), %xmm$reg_num\n"; } - $code .= "add $alloc_size, %rsp\n"; + $code .= "add \$$alloc_size, %rsp\n"; } # Restore any general purpose registers that were saved earlier.