[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 #48 from Luke Kenneth Casson Leighton <lkcl at lkcl.net> ---
(In reply to Jacob Lifshay from comment #43)

> oh, it is how to handle m.If/Switch/Case with part-wise assignment. it uses
> the approach of translating all but the final assignment to just data-flow
> ahead-of-time, letting yosys handle the final assignment translation.

ok i suspected you might have jumped context, thank tou for confirming.
i was referring to an internal use of a Type 2 (dsl.Module) Switch for
a submodule, *not* to the implementation *of* PartitionedSignal.__Switch__

the strategy there for PartotionedSignal.__Switch__ is, well, quite
mindbending, and also out of scope for thiw bugreport. for
completeness however here is the strategy, which should
be discussed there, not here
https://bugs.libre-soc.org/show_bug.cgi?id=458#c10
> iirc nothing actually uses PartitionedSignal yet, so a API break to using
> SimdLayout shouldn't be as big a problem.

except that there may be good reasons why other people (other customers
whom i cannot reveal or discuss because it is commercially confidential)
who might wish to use PartitionedSignal *as it is* i.e. who may not
wish to use Elwidths or who may wish to define their own
adapter or class on *top* of PartitionedSignal, or whom may
wish to have *all* the mask points available not the subset as
fictated by Elwidths.

basically i am saying it is too early to go destroying the
existing PartitionedSignal API and wholesale converting all
of the unit tests to ElwidthPartitionedSignal.

one *small* step at a time.

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