diff options
| author | redfish <redfish@galactica.pw> | 2016-05-25 15:53:29 -0400 |
|---|---|---|
| committer | redfish <redfish@galactica.pw> | 2016-05-25 15:53:29 -0400 |
| commit | 95be58ef0e17a2f1367bcbfab6ed43766ea838be (patch) | |
| tree | 7936d34ea6cc597a8b8eb3974ec02bc0d527c438 | |
| parent | Merge pull request #842 (diff) | |
| download | monero-95be58ef0e17a2f1367bcbfab6ed43766ea838be.tar.xz | |
crypto: slow-hash: disable mul implemented in asm for ARM
The implementation of mul in asm breaks 'slow-hash' test when built with
GCC 6.1.1. Disable this implementation in favor of plain C until it is
fixed.
| -rw-r--r-- | src/crypto/slow-hash.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/src/crypto/slow-hash.c b/src/crypto/slow-hash.c index 4efa8af6c..6e03be4d4 100644 --- a/src/crypto/slow-hash.c +++ b/src/crypto/slow-hash.c @@ -683,7 +683,8 @@ static void (*const extra_hashes[4])(const void *, size_t, char *) = { #include "aesb.c" -/* The asm corresponds to this C code +#ifndef ARM_MUL_IMPL_ASM +/* The asm corresponds to this C code */ #define SHORT uint32_t #define LONG uint64_t @@ -714,7 +715,8 @@ void mul(const uint8_t *ca, const uint8_t *cb, uint8_t *cres) { res[3] = t.tmp[2]; res[0] = t.tmp[6]; res[1] = t.tmp[7]; -} */ +} +#else // ARM_MUL_IMPL_ASM (TODO: this fails hash-slow test with GCC 6.1.1) /* Can work as inline, but actually runs slower. Keep it separate */ #define mul(a, b, c) cn_mul128(a, b, c) @@ -749,6 +751,7 @@ __asm__ __volatile__( : [A]"r"(aa[1]), [a]"r"(aa[0]), [B]"r"(bb[1]), [b]"r"(bb[0]), [r]"r"(r) : "cc", "memory"); } +#endif // ARM_MUL_IMPL_ASM STATIC INLINE void sum_half_blocks(uint8_t* a, const uint8_t* b) { |