[Libre-soc-bugs] [Bug 325] create POWER9 TRAP pipeline

[bugzilla-daemon at libre-soc.org]

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

--- Comment #127 from Luke Kenneth Casson Leighton <lkcl at lkcl.net> ---
this comment has been removed:

@@ -239,24 +211,6 @@ class Driver(Elaboratable):
                 comb += [
                     Assert(msr_o.ok),
                     Assert(nia_o.ok),
-                    ]
-
-                # Note: going through the spec pseudo-code, line-by-line,
-                # in order, with these assertions.  idea is: compare
-                # *directly* against the pseudo-code.  therefore, leave
-                # numbering in (from pseudo-code) and add *comments* about
-                # which field it is (3 == HV etc.)

its purpose is to explain why the strategy of how the code has been written
was chosen.

removing it leaves people confused as to why the code is written the way
that it is.

plus, a reviewer will think, "why is this code written in this fashion??
it's so laborious, plenty of opportunities to quotes optimise quotes",
not understanding that those optimisations *interfere* with clarity.

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