[Libre-soc-bugs] [Bug 713] PartitionedSignal enhancement to add partition-context-aware lengths

[bugzilla-daemon at libre-soc.org]

https://bugs.libre-soc.org/show_bug.cgi?id=713

--- Comment #120 from Luke Kenneth Casson Leighton <lkcl at lkcl.net> ---
(In reply to Jacob Lifshay from comment #119)

> Done. I changed bmask to be such that bmask & (1 << i) is set if partition
> index i is always padding. Note that there is one more partition than the
> number of ppoints, since ppoints separate partitions, not are partitions.
> 
> Demo manually worked out in comments:
> https://git.libre-soc.org/?p=ieee754fpu.git;a=blob;f=src/ieee754/part/
> layout_experiment.py;h=6c32022ec750433ff5adca21f4345cfca51181e2;hb=HEAD#l324


darn it closed the tab by mistake lost a reply. try again.

this is really good. saved several days, i would have taken that long to
work this out. thank you.

the demo / test is really clear, shows what is going on.

also i noticed you did incremental commits, really appreciated.
each one took like only 30 seconds to review, really easy to
understand. i am under enormous pressure at the moment so this
was important.

Slice and Part next. will find the bugreport.

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