Implementation notes: amd64, speed2supercop, crypto_hash/bmw512

Compiler output

update.c: update.c:83:84: warning: '&' within '^' [-Wbitwise-op-parentheses]
update.c:         hashState512_(state).LastPart[LastByte] = hashState512_(state).LastPart[LastByte] & (0xff << (PadOnePosition + 1) )\
update.c:                                                   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
update.c: update.c:83:84: note: place parentheses around the '&' expression to silence this warning
update.c:         hashState512_(state).LastPart[LastByte] = hashState512_(state).LastPart[LastByte] & (0xff << (PadOnePosition + 1) )\
update.c:                                                   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
update.c: 1 warning generated.
try.c: /usr/bin/ld: libcrypto_hash_bmw512.a(Compress.o): warning: relocation in read-only section `.eh_frame'
try.c: /usr/bin/ld: warning: creating DT_TEXTREL in a PIE
try.c: /usr/bin/ld: libcrypto_hash_bmw512.a(Compress.o): warning: relocation in read-only section `.eh_frame'
try.c: /usr/bin/ld: warning: creating DT_TEXTREL in a PIE
measure.c: /usr/bin/ld: libcrypto_hash_bmw512.a(Compress.o): warning: relocation in read-only section `.eh_frame'
measure.c: /usr/bin/ld: warning: creating DT_TEXTREL in a PIE

Compiler output

update.c: update.c: In function 'Hash':
update.c: update.c:83:91: warning: suggest parentheses around arithmetic in operand of '^' [-Wparentheses]
update.c:    83 |         hashState512_(state).LastPart[LastByte] = hashState512_(state).LastPart[LastByte] & (0xff << (PadOnePosition + 1) )\
update.c:       |                                                                                           ^
update.c: update.c:127:9: warning: array subscript -2 is outside array bounds of 'u_int64_t[16]' {aka 'long unsigned int[16]'} [-Warray-bounds=]
update.c:   127 |         Compress512(data64, data64 +16, (hashState *) (CONST64final-2)); // TODO: This is a HACK...
update.c:       |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
update.c: update.c:113:50: note: at offset -16 into object 'CONST64final' of size 128
update.c:   113 |         u_int64_t __attribute__ ((aligned (16))) CONST64final[16] =
update.c:       |                                                  ^~~~~~~~~~~~
try.c: /usr/bin/ld: libcrypto_hash_bmw512.a(Compress.o): warning: relocation in read-only section `.eh_frame'
try.c: /usr/bin/ld: warning: creating DT_TEXTREL in a PIE
try.c: /usr/bin/ld: libcrypto_hash_bmw512.a(Compress.o): warning: relocation in read-only section `.eh_frame'
try.c: /usr/bin/ld: warning: creating DT_TEXTREL in a PIE
measure.c: /usr/bin/ld: libcrypto_hash_bmw512.a(Compress.o): warning: relocation in read-only section `.eh_frame'
measure.c: /usr/bin/ld: warning: creating DT_TEXTREL in a PIE

Compiler output

update.c: update.c: In function 'Hash':
update.c: update.c:83:91: warning: suggest parentheses around arithmetic in operand of '^' [-Wparentheses]
update.c:    83 |         hashState512_(state).LastPart[LastByte] = hashState512_(state).LastPart[LastByte] & (0xff << (PadOnePosition + 1) )\
update.c:       |                                                                                           ^
try.c: /usr/bin/ld: libcrypto_hash_bmw512.a(Compress.o): warning: relocation in read-only section `.eh_frame'
try.c: /usr/bin/ld: warning: creating DT_TEXTREL in a PIE
try.c: /usr/bin/ld: libcrypto_hash_bmw512.a(Compress.o): warning: relocation in read-only section `.eh_frame'
try.c: /usr/bin/ld: warning: creating DT_TEXTREL in a PIE
measure.c: /usr/bin/ld: libcrypto_hash_bmw512.a(Compress.o): warning: relocation in read-only section `.eh_frame'
measure.c: /usr/bin/ld: warning: creating DT_TEXTREL in a PIE

Compiler output

BlueMidnightWish.c: BlueMidnightWish.c:1182:29: warning: unused variable 'tdat32' [-Wunused-variable]
BlueMidnightWish.c:         u_int32_t *data32, *p256, *tdat32, *tp256;
BlueMidnightWish.c:                                    ^
BlueMidnightWish.c: BlueMidnightWish.c:1182:38: warning: unused variable 'tp256' [-Wunused-variable]
BlueMidnightWish.c:         u_int32_t *data32, *p256, *tdat32, *tp256;
BlueMidnightWish.c:                                             ^
BlueMidnightWish.c: BlueMidnightWish.c:1188:69: warning: unused variable 't256_22' [-Wunused-variable]
BlueMidnightWish.c:     u_int32_t t256_16, t256_17, t256_18, t256_19, t256_20, t256_21, t256_22, t256_23;
BlueMidnightWish.c:                                                                     ^
BlueMidnightWish.c: BlueMidnightWish.c:1188:78: warning: unused variable 't256_23' [-Wunused-variable]
BlueMidnightWish.c:     u_int32_t t256_16, t256_17, t256_18, t256_19, t256_20, t256_21, t256_22, t256_23;
BlueMidnightWish.c:                                                                              ^
BlueMidnightWish.c: BlueMidnightWish.c:1189:15: warning: unused variable 't256_24' [-Wunused-variable]
BlueMidnightWish.c:     u_int32_t t256_24, t256_25, t256_26, t256_27, t256_28, t256_29, t256_30, t256_31;
BlueMidnightWish.c:               ^
BlueMidnightWish.c: BlueMidnightWish.c:1189:24: warning: unused variable 't256_25' [-Wunused-variable]
BlueMidnightWish.c:     u_int32_t t256_24, t256_25, t256_26, t256_27, t256_28, t256_29, t256_30, t256_31;
BlueMidnightWish.c:                        ^
BlueMidnightWish.c: BlueMidnightWish.c:1189:33: warning: unused variable 't256_26' [-Wunused-variable]
BlueMidnightWish.c:     u_int32_t t256_24, t256_25, t256_26, t256_27, t256_28, t256_29, t256_30, t256_31;
BlueMidnightWish.c:                                 ^
BlueMidnightWish.c: BlueMidnightWish.c:1189:42: warning: unused variable 't256_27' [-Wunused-variable]
BlueMidnightWish.c:     u_int32_t t256_24, t256_25, t256_26, t256_27, t256_28, t256_29, t256_30, t256_31;
BlueMidnightWish.c:                                          ^
BlueMidnightWish.c: BlueMidnightWish.c:1189:51: warning: unused variable 't256_28' [-Wunused-variable]
BlueMidnightWish.c: ...

Compiler output

BlueMidnightWish.c: BlueMidnightWish.c: In function 'Update':
BlueMidnightWish.c: BlueMidnightWish.c:1200:78: warning: unused variable 't512_31' [-Wunused-variable]
BlueMidnightWish.c:  1200 |     u_int64_t t512_24, t512_25, t512_26, t512_27, t512_28, t512_29, t512_30, t512_31;
BlueMidnightWish.c:       |                                                                              ^~~~~~~
BlueMidnightWish.c: BlueMidnightWish.c:1200:69: warning: unused variable 't512_30' [-Wunused-variable]
BlueMidnightWish.c:  1200 |     u_int64_t t512_24, t512_25, t512_26, t512_27, t512_28, t512_29, t512_30, t512_31;
BlueMidnightWish.c:       |                                                                     ^~~~~~~
BlueMidnightWish.c: BlueMidnightWish.c:1200:60: warning: unused variable 't512_29' [-Wunused-variable]
BlueMidnightWish.c:  1200 |     u_int64_t t512_24, t512_25, t512_26, t512_27, t512_28, t512_29, t512_30, t512_31;
BlueMidnightWish.c:       |                                                            ^~~~~~~
BlueMidnightWish.c: BlueMidnightWish.c:1200:51: warning: unused variable 't512_28' [-Wunused-variable]
BlueMidnightWish.c:  1200 |     u_int64_t t512_24, t512_25, t512_26, t512_27, t512_28, t512_29, t512_30, t512_31;
BlueMidnightWish.c:       |                                                   ^~~~~~~
BlueMidnightWish.c: BlueMidnightWish.c:1200:42: warning: unused variable 't512_27' [-Wunused-variable]
BlueMidnightWish.c:  1200 |     u_int64_t t512_24, t512_25, t512_26, t512_27, t512_28, t512_29, t512_30, t512_31;
BlueMidnightWish.c:       |                                          ^~~~~~~
BlueMidnightWish.c: BlueMidnightWish.c:1200:33: warning: unused variable 't512_26' [-Wunused-variable]
BlueMidnightWish.c:  1200 |     u_int64_t t512_24, t512_25, t512_26, t512_27, t512_28, t512_29, t512_30, t512_31;
BlueMidnightWish.c:       |                                 ^~~~~~~
BlueMidnightWish.c: BlueMidnightWish.c:1200:24: warning: unused variable 't512_25' [-Wunused-variable]
BlueMidnightWish.c:  1200 |     u_int64_t t512_24, t512_25, t512_26, t512_27, t512_28, t512_29, t512_30, t512_31;
BlueMidnightWish.c:       |                        ^~~~~~~
BlueMidnightWish.c: BlueMidnightWish.c:1200:15: warning: unused variable 't512_24' [-Wunused-variable]
BlueMidnightWish.c:  1200 |     u_int64_t t512_24, t512_25, t512_26, t512_27, t512_28, t512_29, t512_30, t512_31;
BlueMidnightWish.c:       |               ^~~~~~~
BlueMidnightWish.c: ...

Compiler output

BlueMidnightWish.c: BlueMidnightWish.c:650:3: warning: misleading indentation; statement is not part of the previous 'else' [-Wmisleading-indentation]
BlueMidnightWish.c:                 p512   = hashState512(state)->DoublePipe;
BlueMidnightWish.c:                 ^
BlueMidnightWish.c: BlueMidnightWish.c:648:2: note: previous statement is here
BlueMidnightWish.c:         else
BlueMidnightWish.c:         ^
BlueMidnightWish.c: BlueMidnightWish.c:688:84: warning: '&' within '^' [-Wbitwise-op-parentheses]
BlueMidnightWish.c:         hashState512(state)->LastPart[LastByte] = hashState512(state)->LastPart[LastByte] & (0xff << (PadOnePosition + 1) )\
BlueMidnightWish.c:                                                   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
BlueMidnightWish.c: BlueMidnightWish.c:688:84: note: place parentheses around the '&' expression to silence this warning
BlueMidnightWish.c:         hashState512(state)->LastPart[LastByte] = hashState512(state)->LastPart[LastByte] & (0xff << (PadOnePosition + 1) )\
BlueMidnightWish.c:                                                   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
BlueMidnightWish.c: 2 warnings generated.

Compiler output

BlueMidnightWish.c: BlueMidnightWish.c: In function 'Update':
BlueMidnightWish.c: BlueMidnightWish.c:648:9: warning: this 'else' clause does not guard... [-Wmisleading-indentation]
BlueMidnightWish.c:   648 |         else
BlueMidnightWish.c:       |         ^~~~
BlueMidnightWish.c: BlueMidnightWish.c:650:17: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'else'
BlueMidnightWish.c:   650 |                 p512   = hashState512(state)->DoublePipe;
BlueMidnightWish.c:       |                 ^~~~
BlueMidnightWish.c: BlueMidnightWish.c: In function 'Final':
BlueMidnightWish.c: BlueMidnightWish.c:688:91: warning: suggest parentheses around arithmetic in operand of '^' [-Wparentheses]
BlueMidnightWish.c:   688 |         hashState512(state)->LastPart[LastByte] = hashState512(state)->LastPart[LastByte] & (0xff << (PadOnePosition + 1) )\
BlueMidnightWish.c:       |                                                                                           ^

Compiler output

BlueMidnightWish.c: BlueMidnightWish.c:988:84: warning: '&' within '^' [-Wbitwise-op-parentheses]
BlueMidnightWish.c:         hashState512_(state).LastPart[LastByte] = hashState512_(state).LastPart[LastByte] & (0xff << (PadOnePosition + 1) )\
BlueMidnightWish.c:                                                   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
BlueMidnightWish.c: BlueMidnightWish.c:988:84: note: place parentheses around the '&' expression to silence this warning
BlueMidnightWish.c:         hashState512_(state).LastPart[LastByte] = hashState512_(state).LastPart[LastByte] & (0xff << (PadOnePosition + 1) )\
BlueMidnightWish.c:                                                   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
BlueMidnightWish.c: 1 warning generated.

Compiler output

BlueMidnightWish.c: BlueMidnightWish.c: In function 'Hash':
BlueMidnightWish.c: BlueMidnightWish.c:988:91: warning: suggest parentheses around arithmetic in operand of '^' [-Wparentheses]
BlueMidnightWish.c:   988 |         hashState512_(state).LastPart[LastByte] = hashState512_(state).LastPart[LastByte] & (0xff << (PadOnePosition + 1) )\
BlueMidnightWish.c:       |                                                                                           ^

Compiler output

BlueMidnightWish.c: BlueMidnightWish.c:390:44: warning: incompatible pointer types passing 'u_int64_t *' (aka 'unsigned long *') to parameter of type 'const __m128i *' [-Wincompatible-pointer-types]
BlueMidnightWish.c:         _mm_store_si128(&p512[00], _mm_load_si128(&hashState512(state)->DoublePipe[0]));
BlueMidnightWish.c:                                                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
BlueMidnightWish.c: /usr/lib/llvm-16/lib/clang/16/include/emmintrin.h:3354:31: note: passing argument to parameter '__p' here
BlueMidnightWish.c: _mm_load_si128(__m128i const *__p) {
BlueMidnightWish.c:                               ^
BlueMidnightWish.c: BlueMidnightWish.c:390:18: warning: incompatible pointer types passing 'u_int64_t *' (aka 'unsigned long *') to parameter of type '__m128i *' [-Wincompatible-pointer-types]
BlueMidnightWish.c:         _mm_store_si128(&p512[00], _mm_load_si128(&hashState512(state)->DoublePipe[0]));
BlueMidnightWish.c:                         ^~~~~~~~~
BlueMidnightWish.c: /usr/lib/llvm-16/lib/clang/16/include/emmintrin.h:3803:68: note: passing argument to parameter '__p' here
BlueMidnightWish.c: static __inline__ void __DEFAULT_FN_ATTRS _mm_store_si128(__m128i *__p,
BlueMidnightWish.c:                                                                    ^
BlueMidnightWish.c: BlueMidnightWish.c:391:44: warning: incompatible pointer types passing 'u_int64_t *' (aka 'unsigned long *') to parameter of type 'const __m128i *' [-Wincompatible-pointer-types]
BlueMidnightWish.c:         _mm_store_si128(&p512[02], _mm_load_si128(&hashState512(state)->DoublePipe[2]));
BlueMidnightWish.c:                                                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
BlueMidnightWish.c: /usr/lib/llvm-16/lib/clang/16/include/emmintrin.h:3354:31: note: passing argument to parameter '__p' here
BlueMidnightWish.c: _mm_load_si128(__m128i const *__p) {
BlueMidnightWish.c:                               ^
BlueMidnightWish.c: BlueMidnightWish.c:391:18: warning: incompatible pointer types passing 'u_int64_t *' (aka 'unsigned long *') to parameter of type '__m128i *' [-Wincompatible-pointer-types]
BlueMidnightWish.c:         _mm_store_si128(&p512[02], _mm_load_si128(&hashState512(state)->DoublePipe[2]));
BlueMidnightWish.c:                         ^~~~~~~~~
BlueMidnightWish.c: /usr/lib/llvm-16/lib/clang/16/include/emmintrin.h:3803:68: note: passing argument to parameter '__p' here
BlueMidnightWish.c: static __inline__ void __DEFAULT_FN_ATTRS _mm_store_si128(__m128i *__p,
BlueMidnightWish.c:                                                                    ^
BlueMidnightWish.c: BlueMidnightWish.c:392:44: warning: incompatible pointer types passing 'u_int64_t *' (aka 'unsigned long *') to parameter of type 'const __m128i *' [-Wincompatible-pointer-types]
BlueMidnightWish.c: ...

Compiler output

BlueMidnightWish.c: BlueMidnightWish.c: In function 'Compress512':
BlueMidnightWish.c: BlueMidnightWish.c:390:51: warning: passing argument 1 of '_mm_load_si128' from incompatible pointer type [-Wincompatible-pointer-types]
BlueMidnightWish.c:   390 |         _mm_store_si128(&p512[00], _mm_load_si128(&hashState512(state)->DoublePipe[0]));
BlueMidnightWish.c:       |                                                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
BlueMidnightWish.c:       |                                                   |
BlueMidnightWish.c:       |                                                   u_int64_t * {aka long unsigned int *}
BlueMidnightWish.c: In file included from BlueMidnightWish.c:12:
BlueMidnightWish.c: /usr/lib/gcc/x86_64-linux-gnu/13/include/emmintrin.h:698:32: note: expected 'const __m128i *' but argument is of type 'u_int64_t *' {aka 'long unsigned int *'}
BlueMidnightWish.c:   698 | _mm_load_si128 (__m128i const *__P)
BlueMidnightWish.c:       |                 ~~~~~~~~~~~~~~~^~~
BlueMidnightWish.c: BlueMidnightWish.c:390:25: warning: passing argument 1 of '_mm_store_si128' from incompatible pointer type [-Wincompatible-pointer-types]
BlueMidnightWish.c:   390 |         _mm_store_si128(&p512[00], _mm_load_si128(&hashState512(state)->DoublePipe[0]));
BlueMidnightWish.c:       |                         ^~~~~~~~~
BlueMidnightWish.c:       |                         |
BlueMidnightWish.c:       |                         u_int64_t * {aka long unsigned int *}
BlueMidnightWish.c: /usr/lib/gcc/x86_64-linux-gnu/13/include/emmintrin.h:734:27: note: expected '__m128i *' but argument is of type 'u_int64_t *' {aka 'long unsigned int *'}
BlueMidnightWish.c:   734 | _mm_store_si128 (__m128i *__P, __m128i __B)
BlueMidnightWish.c:       |                  ~~~~~~~~~^~~
BlueMidnightWish.c: BlueMidnightWish.c:391:51: warning: passing argument 1 of '_mm_load_si128' from incompatible pointer type [-Wincompatible-pointer-types]
BlueMidnightWish.c:   391 |         _mm_store_si128(&p512[02], _mm_load_si128(&hashState512(state)->DoublePipe[2]));
BlueMidnightWish.c:       |                                                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
BlueMidnightWish.c:       |                                                   |
BlueMidnightWish.c:       |                                                   u_int64_t * {aka long unsigned int *}
BlueMidnightWish.c: /usr/lib/gcc/x86_64-linux-gnu/13/include/emmintrin.h:698:32: note: expected 'const __m128i *' but argument is of type 'u_int64_t *' {aka 'long unsigned int *'}
BlueMidnightWish.c:   698 | _mm_load_si128 (__m128i const *__P)
BlueMidnightWish.c: ...