[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
Wed Oct 6 12:27:00 BST 2021


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

> assuming a layout of:
> SimdLayout({
>     1: unsigned(10),
>     2: unsigned(12),
>     4: unsigned(15),
> })

noooo, different types of signedness down at the lane level is
completely out of the question.

that significantly changes the design and, worse, breaks a fundamental
assumption that the object can be treated as "if it was one thing"
by nmigen Type 1 (ast.*) and Type 2 (dsl.Module) language constructs.

this would be very bad and very complicated, very quickly.

and i don't see any need for it.

therefore, the signedness can in fact be removed, entirely, and nmigen
Shape signed/unsigned flag used.

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

More information about the libre-soc-bugs mailing list