[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 #93 from Luke Kenneth Casson Leighton <lkcl at lkcl.net> ---
(In reply to Jacob Lifshay from comment #91)
> yup, both are needed, but full-simd-width is passed in through a
> scoped-global, not through SimdSignal.__init__ arguments.

no. won't work.  parameter to SimdShape.  basically parameters to
layout() become parameters to SimdShape.

full_simd_width named parameter defaults to None, this is fixed_width
param of layout().  gets put into Shape.width directly.

SimdShape constructor 1st arg, if int, this is vector element width.
if dict, is the layout elwidths thing.
actual width is computed and set in Shape.width


> > 
> > (In reply to Jacob Lifshay from comment #89)
> > 
> > > I do think that having just the XLEN global idea will save us a huge amount
> > > of time, 
> > 
> > yes.  but only if its class is SimdShape which derives from Shape.
> 
> wait...I am planning on XLEN being a type 

too much focus on "types", transferring concepts from rust not
appropriate.

XLEN as a SimdShape, fixed_width=64.

see url at top of bugreport.

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