OpenCores
Issue List
"tv80_core.v" have missing elements in a sensitivity list #9
Closed ClaudeQC opened this issue about 9 years ago
ClaudeQC commented about 9 years ago

In "tv80_core.v", at lines 1333-1334, there are missing elements in the sensitivity list. Below, the patch that correct that problem:

			      +++

--- tv80_core_ss_2015-11-12_1757.v 2015-11-12 17:57:31.712532800 -0500 +++ tv80_core.v 2015-11-12 18:04:43.443226400 -0500 @@ -1330,8 +1330,9 @@ end end

  • always @(/AUTOSENSE/BTR_r or DI_Reg or IncDec_16 or JumpE or PC
  • or RegBusA or RegBusC or SP or tstate)
  • always @( /AUTOSENSE/
  •  		BTR_r or DI_Reg or IncDec_16 or JumpE or PC or RegBusA or
  • 		RegBusC or SP or tstate or ID16_B or PC16_B or SP16_A or SP16_B)
     begin
       if (JumpE == 1'b1 ) 
         begin
    @@ -1370,7 +1371,8 @@ ID16 = RegBusA + ID16_B; PC16 = PC + PC16_B; SP16 = SP16_A + SP16_B;
  • end // always @ *

+

  • end // always @ ...

    always @(/AUTOSENSE/IntCycle or NMICycle or mcycle)

    				      +++

Regards, From a tv80 user.

Claude

ClaudeQC commented about 9 years ago

Well... Seems that this was already reported on "Jan 15, 2011" bug report. ... And this is not a bug. So, forget my bug report.

However, I noticed the following:

ISE V14.7 on a Spartan-3E output warning about missing elements on the sensitivity list.

Quartus V15.1 on a Cyclone-IV do not output such warning.

Claude

ghutchis closed this almost 6 years ago

Assignee
No one
Labels
Bug