riscv-isa-manual icon indicating copy to clipboard operation
riscv-isa-manual copied to clipboard

Published 20 hours ago • verified publisher icon riscv

more chapter re-ordering

Open aamartin0000 opened this issue 11 months ago • 3 comments

Thank you for rearranging the chapters to a more sane ordering. Some suggestions, take 'em or leave 'em:

  • in Ch28, Zbkb should be before Zbkc. Now that the Zbk* descriptions are within the bitmanip chapter, would it be worth mentioning overlaps (Zbkb in Zbb, Zbkc == Zbc)?
  • Ch33 (scalar crypto) should be before Ch32 (vector crypto)
  • please add sample code for use of (scalar and vector) crypto instructions

aamartin0000 avatar Mar 22 '24 16:03 aamartin0000

@aswaterman I can do this if you agree with it.

kersten1 avatar Mar 29 '24 21:03 kersten1

I think some of the reorg might’ve already been done, but the rest makes sense, so go for it. The sample code is a longer-term thing we don’t need to worry about now.

aswaterman avatar Mar 29 '24 22:03 aswaterman

Agreed. Please close this issue when the pull request is opened.

Perhaps members of the crypto SIG/TG can contribute sample (scalar, vector) crypto code when the time is right.

aamartin0000 avatar Mar 29 '24 22:03 aamartin0000

Closing as reordering patch applied.

wmat avatar Apr 05 '24 11:04 wmat