[Libre-soc-bugs] [Bug 736] SimdSignal's integration with nmigen needs to handle first args not being SimdSignals

[bugzilla-daemon at libre-soc.org]

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

--- Comment #3 from Jacob Lifshay <programmerjake at gmail.com> ---
(In reply to Luke Kenneth Casson Leighton from comment #2)
> as in, rather than:
> 
> s = SimdSignal(...)
> Cat(Const(...), s)
> 
> it would be e.g.:
> s = SimdSignal(...)
> c = SimdSignal(...)
> comb += c.eq(Const(...))
> Cat(c, s)
> 
> which "avoids" the problem that i think might turn out
> to be unique to Cat()

nope, it also occurs in Mux and Part.

s1 = Signal()
s2 = SimdSignal()
Mux(s1, Const(), s2)
Part(Const(), s2, 3, 5)

For Cat, we look for the most-derived subclass and use that. more when I have
time.

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