yuzu/mbedtls
1
0
Fork 0
mirror of https://github.com/yuzu-emu/mbedtls.git synced 2025-06-05 11:58:25 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix aarch64 assembly for bignum multiplication

Browse source 
Add memory constraints to the aarch64 inline assembly in MULADDC_STOP.
This fixes an issue where Clang 12 and 13 were generating
non-functional code on aarch64 platforms. See #4962, #4943
for further details.

Signed-off-by: David Horstmann <david.horstmann@arm.com>
This commit is contained in:
David Horstmann 2021-09-22 18:15:51 +01:00
parent 02e17c0aa5
commit a23be22308
2 changed files with 10 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
ChangeLog.d/muladdc-aarch64-memory.txt Normal file
View file

@ -0,0 +1,4 @@
Bugfix
* Add missing memory constraints in aarch64 inline assembly for
bignum multiplication.
Fixes #4962.

12
include/mbedtls/bn_mul.h
View file

@ -244,18 +244,18 @@
#define MULADDC_CORE \ #define MULADDC_CORE \
"ldr x4, [%2], #8 \n\t" \ "ldr x4, [%2], #8 \n\t" \
"ldr x5, [%1] \n\t" \ "ldr x5, [%1] \n\t" \
"mul x6, x4, %3 \n\t" \ "mul x6, x4, %4 \n\t" \
"umulh x7, x4, %3 \n\t" \ "umulh x7, x4, %4 \n\t" \
"adds x5, x5, x6 \n\t" \ "adds x5, x5, x6 \n\t" \
"adc x7, x7, xzr \n\t" \ "adc x7, x7, xzr \n\t" \
"adds x5, x5, %0 \n\t" \ "adds x5, x5, %0 \n\t" \
"adc %0, x7, xzr \n\t" \ "adc %0, x7, xzr \n\t" \
"str x5, [%1], #8 \n\t" "str x5, [%1], #8 \n\t"
#define MULADDC_STOP \ #define MULADDC_STOP \
: "+r" (c), "+r" (d), "+r" (s) \ : "+r" (c), "+r" (d), "+r" (s), "+m" (*(uint64_t (*)[16]) d) \
: "r" (b) \ : "r" (b), "m" (*(const uint64_t (*)[16]) s) \
: "x4", "x5", "x6", "x7", "cc" \ : "x4", "x5", "x6", "x7", "cc" \
); );
#endif /* Aarch64 */ #endif /* Aarch64 */