aes/asm/bsaes-armv7.pl: relax stack alignment requirement.
authorAndy Polyakov <appro@openssl.org>
Sat, 25 Mar 2017 09:58:57 +0000 (10:58 +0100)
committerAndy Polyakov <appro@openssl.org>
Sun, 26 Mar 2017 16:29:03 +0000 (18:29 +0200)
Even though Apple refers to Procedure Call Standard for ARM Architecture
(AAPCS), they apparently adhere to custom version that doesn't follow
stack alignment constraints in the said standard. [Why or why? If it's
vendor lock-in thing, then it would be like worst spot ever.] And since
bsaes-armv7 relied on standard alignment, it became problematic to
execute the code on iOS.

Reviewed-by: Rich Salz <rsalz@openssl.org>
crypto/aes/asm/bsaes-armv7.pl

index 33295881e1968b2e004e2c60d9d4ddd2a4f7bc69..ba641eb547f68b59e04de2881da1b0271500b3cc 100644 (file)
@@ -1365,7 +1365,7 @@ bsaes_cbc_encrypt:
        vmov    @XMM[4],@XMM[15]                @ just in case ensure that IV
        vmov    @XMM[5],@XMM[0]                 @ and input are preserved
        bl      AES_decrypt
-       vld1.8  {@XMM[0]}, [$fp,:64]            @ load result
+       vld1.8  {@XMM[0]}, [$fp]                @ load result
        veor    @XMM[0], @XMM[0], @XMM[4]       @ ^= IV
        vmov    @XMM[15], @XMM[5]               @ @XMM[5] holds input
        vst1.8  {@XMM[0]}, [$rounds]            @ write output