[libre-riscv-dev] priority encoder replacement

Jacob Lifshay programmerjake at gmail.com
Tue May 19 19:01:59 BST 2020


On Tue, May 19, 2020, 10:57 Luke Kenneth Casson Leighton <lkcl at lkcl.net>
wrote:

> On Tue, May 19, 2020 at 6:47 PM Michael Nolan <mtnolan2640 at gmail.com>
> wrote:
> > I also looked at the differences between nmutil.clz and nmigen's
> > PriorityEncoder, and summarized them in
> > https://bugs.libre-soc.org/show_bug.cgi?id=326
>
> yep good call, closed and commented nmigen.clz given that it's better.
>
> have to remember that because PriorityEncoder gets used in a few places.


If clz is strictly better, replace PriorityEncoder with clz, or just change
clz to match PriorityEncoder's interface and rename it PriorityEncoder,
deleting the old one.

Jacob


More information about the libre-riscv-dev mailing list