[libre-riscv-dev] [Bug 173] dynamic partitioned "shift"

bugzilla-daemon at libre-riscv.org bugzilla-daemon at libre-riscv.org
Wed Feb 12 18:58:15 GMT 2020


--- Comment #8 from Michael Nolan <mtnolan2640 at gmail.com> ---
(In reply to Luke Kenneth Casson Leighton from comment #5)

> the yosys graph is wonderful: like a dog munched your spaghetti bolognaise
> dinner then threw it up when you held the dog up at waist-level and did a
> nice pirouette :)

ROFL, that's far more creative than most of my insults usually are. 
I would have just said it looks like sh*t.

(In reply to Luke Kenneth Casson Leighton from comment #6)
> wait... no... yes, it looks like you've already done that (to some extent),
> at least limited the range of the matrix partial-results.
> however what's missing is the truncation of B to match the length of the
> output:
>                 comb += matrix[i][j].eq(a << b)
>                 start = end
> should be something like:
>                 outwid = matrix[i][j].shape()[0]
>                 bwid = math.ceil(math.log2(outwid + 1))
>                 comb += matrix[i][j].eq(a << b[:bwid])
>                 start = end
> or something like that.  there's almost certainly at least two off-by-one
> bugs in that :)

I suspect yosys is doing some of this already, since the size of my results are
limited, but I can give it a try.

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

More information about the libre-riscv-dev mailing list