[Libre-soc-bugs] [Bug 318] fix LDSTCompUnit

bugzilla-daemon at libre-soc.org bugzilla-daemon at libre-soc.org
Mon Jun 22 12:35:32 BST 2020


--- Comment #30 from Luke Kenneth Casson Leighton <lkcl at lkcl.net> ---
i just noticed also that CompLDSTOpSubset has all these fields:

                  ('is_32bit', 1),
                  ('is_signed', 1),
                  ('data_len', 4),
                  ('byte_reverse', 1),
                  ('sign_extend', 1),
                  ('update', 1))

update is done, data_len is respected, byte_reverse is done: that leaves
any changes needed from is_32bit, is_signed and sign_extend.  i'd expect
sign_extend in particular to also have to take is_32bit into consideration.

i'm tempted to suggest moving this data-manipulation into a separate module
in order to keep things looking "clean" in the section covering
"PortInterface Connections"

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

More information about the libre-soc-bugs mailing list