Skip to content
This repository has been archived by the owner on Jul 23, 2024. It is now read-only.

[scala crypto] parameter type of sm3 intrinsic #11

Open
Xinlong-Wu opened this issue Apr 4, 2022 · 0 comments
Open

[scala crypto] parameter type of sm3 intrinsic #11

Xinlong-Wu opened this issue Apr 4, 2022 · 0 comments

Comments

@Xinlong-Wu
Copy link

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?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant