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

bugzilla-daemon at libre-soc.org bugzilla-daemon at libre-soc.org
Sat Oct 9 01:03:14 BST 2021


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

> oh, i can do that...i'll convert PartitionedSignal + unit tests to use
> SimdLayout.

whoawhoa hang on, only do that if you're happy that SimdLayout should derive 
from ast.Shape, that layout() should take a dict *only* (not have
because, obviously, by deriving from ast.Shape() SimdLayout *already has*
a signed member, and *only* if you can guarantee to have it 100% completed
within one day.

the reason is because if not i have to completely stop all ongoing
work on PartitionedSignal because it becomes too much to merge
new submodules into the altered code.

(i am currently investigating how to add __getattr__, Part, and Slice,
all of which are interrelated)

welcome to planning on large complex projects, sigh.

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

More information about the libre-soc-bugs mailing list