[libre-riscv-dev] [Bug 316] bperm TODO

[bugzilla-daemon at libre-soc.org]

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

Yehowshua <yimmanuel3 at gatech.edu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |yimmanuel3 at gatech.edu

--- Comment #2 from Yehowshua <yimmanuel3 at gatech.edu> ---
Change line 13 to 
``m.d.comb += self.perm.eq(0)``

You don't need to use ``Cat`` and or ``Repo``. nMigen knows how to 0 pad and
sign extend.

Now you'll get another issue - namely, that rs isn't define...

What is Bpermd supposed to do?
You should put a Python class comment as shown below to help readers
understand.

```
class Bpermd(Elaboratable):
"""This class does X,Y, and Z"""
    def __init__(self, width):
        self.perm = Signal(width)
```

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