[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 #103 from Luke Kenneth Casson Leighton <lkcl at lkcl.net> ---
cpart_wid = max(lane_shapes.values())
width = cpart_wid * part_count

replaced with:

+    cpart_wid = 0
+    width = 0
+    for i, lwid in lane_shapes.items():
+        required_width = lwid * vec_el_counts[i]
+        print("     required width", cpart_wid, i, lwid, required_width)
+        if required_width > width:
+            cpart_wid = lwid
+            width = required_width

commit 316c02716c365eee298ca3da5a65c3be58b57a1e (HEAD -> master)
Author: Luke Kenneth Casson Leighton <lkcl at lkcl.net>
Date:   Wed Oct 13 15:44:19 2021 +0100

    fix issue where width was being computed based on 2 maximum values
    actually needed is to multiply the number of elements by the width of
    an element and use that to determine which is greater

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