rvkrypto/rvkrypto-fips

[scala crypto] parameter type of sm3 intrinsic

Opened this issue · 0 comments

Hi, all

We notice that the definition of intrinsic for sm3 is

static inline long _rv_sm3p1(long rs1)
	{ return _RVK_INTRIN_IMPL(sm3p1)(rs1); }				//	SM3P1

while the operation defined in the specification is
image

We found that sm3p1 only use low 32 bit of rs1 in both rv32 and rv64, but the type of parameter rs1 in intrinsic is long.

so, what about we change the type of rs1 into uint32_t?