more comments for TestIssuer when adding predication
authorLuke Kenneth Casson Leighton <lkcl@lkcl.net>
Fri, 19 Mar 2021 23:40:38 +0000 (23:40 +0000)
committerLuke Kenneth Casson Leighton <lkcl@lkcl.net>
Fri, 19 Mar 2021 23:40:38 +0000 (23:40 +0000)
src/soc/simple/issuer.py

index 805940c7fa363efc950a1f4e654581fba4701a3f..2d02ea1a348905e1d0c8dbbe19f16c80164c173b 100644 (file)
@@ -469,6 +469,9 @@ class TestIssuerInternal(Elaboratable):
                 #    from self.srcmask and self.dstmask
                 #    https://bugs.libre-soc.org/show_bug.cgi?id=617#c3
                 #    but still without exceeding VL in either case
+                # IMPORTANT: when changing src/dest step, have to
+                # jump to m.next = "DECODE_SV" to deal with the change in
+                # SVSTATE
                 comb += exec_insn_valid_i.eq(1) # trigger execute
                 with m.If(exec_insn_ready_o):   # execute acknowledged us
                     m.next = "EXECUTE_WAIT"