[libre-riscv-dev] [Bug 60] N-stage 64-bit multiplier pipeline needed (signed/unsigned)

bugzilla-daemon at libre-riscv.org bugzilla-daemon at libre-riscv.org
Sat May 18 05:25:02 BST 2019


http://bugs.libre-riscv.org/show_bug.cgi?id=60

--- Comment #16 from Luke Kenneth Casson Leighton <lkcl at lkcl.net> ---
(In reply to Jacob Lifshay from comment #15)
> I was thinking it might be a good idea to write an update for croudsupply
> describing the implementation of the multiplier and how I arrived at the
> code I have now from initial principles.

that's a really good idea.  you have access to the repo.
https://git.libre-riscv.org/?p=crowdsupply.git;a=summary

good responses, good justifications for what you did (on previous comment),
use the answers you gave to create comments.

in particular make sure that the recursive case is commented, as it wasn't
clear that the function *is* used recursively! (no comments to that effect,
i didn't notice).

the singleton thing is so unsafe, it causes so many problems and wastes
so much time trying to track it down, it's just not worth the hassle of
trying to save 1 lines of code.  please use the pattern that i provided
instead.

-- 
You are receiving this mail because:
You are on the CC list for the bug.


More information about the libre-riscv-dev mailing list