Skip to content

Commit 366789e

Browse files
mkannwischerhanno-becker
authored andcommitted
mly_sys_cap: Unconditionally define all sys_cap enum entries
Currently the CBMC proofs for the 3 native aarch64 versions of Keccak using SHA-3 instructions are failing as the MLK_SYS_CAP_SHA3 symbol is not defined on an x86 system. This commit fixes this by defining the mlk_sys_cap entries unconditional. Signed-off-by: Matthias J. Kannwischer <matthias@kannwischer.eu>
1 parent a4cb668 commit 366789e

File tree

1 file changed

+3
-8
lines changed

1 file changed

+3
-8
lines changed

mlkem/src/sys.h

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -217,15 +217,10 @@
217217
/* System capability enumeration */
218218
typedef enum
219219
{
220-
#if defined(MLK_SYS_X86_64)
221-
MLK_SYS_CAP_AVX2
222-
#elif defined(MLK_SYS_AARCH64)
220+
/* x86_64 */
221+
MLK_SYS_CAP_AVX2,
222+
/* AArch64 */
223223
MLK_SYS_CAP_SHA3
224-
#else
225-
/* C90 does not allow empty enums, so use a dummy value
226-
* for architectures other than AArch64 and x86_64. */
227-
MLK_SYS_CAP_DUMMY
228-
#endif
229224
} mlk_sys_cap;
230225

231226
#if !defined(MLK_CONFIG_CUSTOM_CAPABILITY_FUNC)

0 commit comments

Comments
 (0)