From: Luke Kenneth Casson Leighton <lkcl@lkcl.net>
Date: Tue, 19 May 2020 17:55:25 +0000 (+0100)
Subject: minor whitespace, comment clz
X-Git-Tag: 24jan2021_ls180~59
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=67b05c351cf81fd606f0866b9d43312d551a0ae1;p=nmutil.git

minor whitespace, comment clz
---

diff --git a/src/nmutil/clz.py b/src/nmutil/clz.py
index 2c40988..cbf3397 100644
--- a/src/nmutil/clz.py
+++ b/src/nmutil/clz.py
@@ -1,6 +1,8 @@
 from nmigen import Module, Signal, Elaboratable, Cat, Repl
 import math
-""" TODO: replace this module with PriorityEncoder
+""" This module is much more efficient than PriorityEncoder
+    although it is functionally identical.
+    see https://bugs.libre-soc.org/show_bug.cgi?id=326
 """
 
 class CLZ(Elaboratable):
@@ -48,8 +50,7 @@ class CLZ(Elaboratable):
                 left, lv = pairs[i+1]
                 right, rv = pairs[i]
                 width = right.width + 1
-                new_pair = Signal(width, name="cnt_%d_%d" %
-                                  (iteration, i))
+                new_pair = Signal(width, name="cnt_%d_%d" % (iteration, i))
                 if rv == lv:
                     with m.If(left[-1] == 1):
                         with m.If(right[-1] == 1):
@@ -64,7 +65,6 @@ class CLZ(Elaboratable):
                     with m.Else():
                         comb += new_pair.eq(left)
 
-
                 ret.append((new_pair, lv+rv))
         return ret
 
@@ -82,4 +82,3 @@ class CLZ(Elaboratable):
 
         return m
 
-