Skip to content

Conversation

@AFOliveira
Copy link
Collaborator

In B extension, although instructions already have the data_independent_timing: true , it also make senses to describe the crypto extensions, since they technically also include those instructions, as described in the ISA Manual :
image

Moreover, I added them in conformity with B extension format [Extension, Subextension, Subextension], e.g.

packw:
...
  definedBy:
    anyOf: [Zbkb, Zk, Zkn, Zks]

@AFOliveira AFOliveira requested a review from dhower-qc as a code owner October 25, 2024 10:13
Signed-off-by: Afonso Oliveira <[email protected]>
@AFOliveira AFOliveira force-pushed the AFOliveira/InstructionsSkeletons branch from c99a9e9 to afefa4c Compare October 25, 2024 10:18
Signed-off-by: Afonso Oliveira <[email protected]>
Signed-off-by: Afonso Oliveira <[email protected]>
Signed-off-by: Afonso Oliveira <[email protected]>
Signed-off-by: Afonso Oliveira <[email protected]>
Signed-off-by: Afonso Oliveira <[email protected]>
Signed-off-by: Afonso Oliveira <[email protected]>
Signed-off-by: Afonso Oliveira <[email protected]>
@AFOliveira AFOliveira changed the title Zk extension Crypto Extensions Oct 25, 2024
Signed-off-by: Afonso Oliveira <[email protected]>
Signed-off-by: Afonso Oliveira <[email protected]>
@AFOliveira AFOliveira force-pushed the AFOliveira/InstructionsSkeletons branch from f40f05f to 7f292fb Compare October 25, 2024 11:28
Copy link
Collaborator

@dhower-qc dhower-qc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@dhower-qc dhower-qc merged commit d647b55 into main Oct 29, 2024
1 check passed
@dhower-qc dhower-qc deleted the AFOliveira/InstructionsSkeletons branch October 29, 2024 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants