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

[bugzilla-daemon at libre-riscv.org]

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

--- 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.