Implementation notes: amd64, hertz, crypto_aead/led80n6t4silcv3

Compiler output

encrypt.c: encrypt.c:70:2: warning: misleading indentation; statement is not part of the previous 'for' [-Wmisleading-indentation]
encrypt.c:    70 |         return RETURN_SUCCESS;
encrypt.c:       |         ^
encrypt.c: encrypt.c:65:5: note: previous statement is here
encrypt.c:    65 |     for(i = 0; i < CRYPTO_ABYTES; i++)
encrypt.c:       |     ^
encrypt.c: 1 warning generated.

Compiler output

encrypt.c: encrypt.c: In function 'crypto_aead_led80n6t4silcv3_ref_timingleaks_decrypt':
encrypt.c: encrypt.c:65:5: warning: this 'for' clause does not guard... [-Wmisleading-indentation]
encrypt.c:    65 |     for(i = 0; i < CRYPTO_ABYTES; i++)
encrypt.c:       |     ^~~
encrypt.c: encrypt.c:70:9: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'for'
encrypt.c:    70 |         return RETURN_SUCCESS;
encrypt.c:       |         ^~~~~~

Compiler output

encrypt.c: encrypt.c:70:2: warning: misleading indentation; statement is not part of the previous 'for' [-Wmisleading-indentation]
encrypt.c:    70 |         return RETURN_SUCCESS;
encrypt.c:       |         ^
encrypt.c: encrypt.c:65:5: note: previous statement is here
encrypt.c:    65 |     for(i = 0; i < CRYPTO_ABYTES; i++)
encrypt.c:       |     ^
encrypt.c: 1 warning generated.
silc.c: silc.c:174:34: warning: implicit conversion from 'int' to 'char' changes value from 128 to -128 [-Wconstant-conversion]
silc.c:   174 |         state = XORDQW(tmpState, SHR(state, 8));
silc.c:       |                 ~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~
silc.c: ./common.h:30:126: note: expanded from macro 'SHR'
silc.c:    30 | #define SHR(x,n)        _mm_shuffle_epi8((x), _mm_set_epi8(127+(n), 126+(n), 125+(n), 124+(n), 123+(n), 122+(n), 121+(n), 120+(n), 119+(n), 118+(n), 117+(n), 116+(n), 115+(n), 114+(n), 113+(n), 112+(n))) // shift to the right
silc.c:       |                                               ~~~~~~~~~~~~                                                                   ^
silc.c: ./common.h:18:43: note: expanded from macro 'XORDQW'
silc.c:    18 | #define XORDQW(x, y)            _mm_xor_si128((x), (y))
silc.c:       |                                                     ^
silc.c: silc.c:174:34: warning: implicit conversion from 'int' to 'char' changes value from 129 to -127 [-Wconstant-conversion]
silc.c:   174 |         state = XORDQW(tmpState, SHR(state, 8));
silc.c:       |                 ~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~
silc.c: ./common.h:30:117: note: expanded from macro 'SHR'
silc.c:    30 | #define SHR(x,n)        _mm_shuffle_epi8((x), _mm_set_epi8(127+(n), 126+(n), 125+(n), 124+(n), 123+(n), 122+(n), 121+(n), 120+(n), 119+(n), 118+(n), 117+(n), 116+(n), 115+(n), 114+(n), 113+(n), 112+(n))) // shift to the right
silc.c:       |                                               ~~~~~~~~~~~~                                                          ^
silc.c: ./common.h:18:43: note: expanded from macro 'XORDQW'
silc.c:    18 | #define XORDQW(x, y)            _mm_xor_si128((x), (y))
silc.c:       |                                                     ^
silc.c: silc.c:174:34: warning: implicit conversion from 'int' to 'char' changes value from 130 to -126 [-Wconstant-conversion]
silc.c:   174 |         state = XORDQW(tmpState, SHR(state, 8));
silc.c:       |                 ~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~
silc.c: ./common.h:30:108: note: expanded from macro 'SHR'
silc.c:    30 | #define SHR(x,n)        _mm_shuffle_epi8((x), _mm_set_epi8(127+(n), 126+(n), 125+(n), 124+(n), 123+(n), 122+(n), 121+(n), 120+(n), 119+(n), 118+(n), 117+(n), 116+(n), 115+(n), 114+(n), 113+(n), 112+(n))) // shift to the right
silc.c:       |                                               ~~~~~~~~~~~~                                                 ^
silc.c: ./common.h:18:43: note: expanded from macro 'XORDQW'
silc.c: ...

Compiler output

encrypt.c: encrypt.c: In function 'crypto_aead_led80n6t4silcv3_vperm_timingleaks_decrypt':
encrypt.c: encrypt.c:65:5: warning: this 'for' clause does not guard... [-Wmisleading-indentation]
encrypt.c:    65 |     for(i = 0; i < CRYPTO_ABYTES; i++)
encrypt.c:       |     ^~~
encrypt.c: encrypt.c:70:9: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'for'
encrypt.c:    70 |         return RETURN_SUCCESS;
encrypt.c:       |         ^~~~~~
silc.c: silc.c: In function 'ae_decrypt':
silc.c: silc.c:194:12: warning: unused variable 'tmpState' [-Wunused-variable]
silc.c:   194 |     dqword tmpState;
silc.c:       |            ^~~~~~~~

Namespace violations

led.o AddConstants T
led.o AddKey T
led.o DEBUG B
led.o FieldMult T
led.o LED D
led.o LED80_enc T
led.o LED_enc T
led.o MixColMatrix R
led.o MixColumn T
led.o ShiftRow T
led.o SubCell T
led.o WORDFILTER R
led.o sbox R
silc.o ae_encrypt T
silc.o ae_init T
silc.o process_ad T
utils.o print_cxt T
utils.o pstate T
utils.o pstate2 T
utils.o xor_bytes T
utils.o xor_bytes2 T

Namespace violations

led.o ANDMASKH_LED R
led.o ANDMASKL_LED R
led.o ANDMASK_LED R
led.o AddConstants T
led.o AddKey T
led.o BitPermuteStep64 T
led.o FormatInputMask1_LED R
led.o FormatInputMask2_LED R
led.o FormatOutput T
led.o FormateInput T
led.o LED80_InitKEY T
led.o LED80_enc T
led.o Mbox1 R
led.o Mbox2 R
led.o Mbox3 R
led.o Mbox4 R
led.o Mbox5 R
led.o Mbox6 R
led.o Mbox7 R
led.o Mbox8 R
led.o MixColumnWithSbox T
led.o NibbleExchange T
led.o ORMASK_LED R
led.o RC_80 R
led.o SHIFTROWH R
led.o SHIFTROWL R
led.o ShiftRow T
led.o rks B
silc.o ae_clear T
silc.o ae_decrypt T
silc.o ae_encrypt T
silc.o ae_init T
silc.o process_ad T
silc.o zap T
silc.o zpp T