You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hey!
Enum should be assignment only by enum type. LRM1800-2017 6.19.4: A cast shall be required for an expression that is assigned to an enum variable where the type of the expression is not equivalent to the enumeration type of the variable.
So in file design/ifu/ifu_ifc_ctl.sv need to be change two lines: assign next_state[1] = (~state[1] & state[0] & ~reset_delayed & miss_f2 & ~goto_idle) | (state[1] & ~reset_delayed & ~mb_empty_mod & ~goto_idle);
for assign next_state[1] = state_t'((~state[1] & state[0] & ~reset_delayed & miss_f2 & ~goto_idle) | (state[1] & ~reset_delayed & ~mb_empty_mod & ~goto_idle));
and assign next_state[0] = (~goto_idle & leave_idle) | (state[0] & ~goto_idle) | (reset_delayed);
for assign next_state[0] = state_t'((~goto_idle & leave_idle) | (state[0] & ~goto_idle) | (reset_delayed));
Like was done in pull request #49
The text was updated successfully, but these errors were encountered:
dawidzim
added a commit
to dawidzim/Cores-SweRV
that referenced
this issue
Jun 26, 2020
Hey!
Enum should be assignment only by enum type. LRM1800-2017 6.19.4:
A cast shall be required for an expression that is assigned to an enum variable where the type of the expression is not equivalent to the enumeration type of the variable.
So in file
design/ifu/ifu_ifc_ctl.sv
need to be change two lines:assign next_state[1] = (~state[1] & state[0] & ~reset_delayed & miss_f2 & ~goto_idle) | (state[1] & ~reset_delayed & ~mb_empty_mod & ~goto_idle);
for
assign next_state[1] = state_t'((~state[1] & state[0] & ~reset_delayed & miss_f2 & ~goto_idle) | (state[1] & ~reset_delayed & ~mb_empty_mod & ~goto_idle));
and
assign next_state[0] = (~goto_idle & leave_idle) | (state[0] & ~goto_idle) | (reset_delayed);
for
assign next_state[0] = state_t'((~goto_idle & leave_idle) | (state[0] & ~goto_idle) | (reset_delayed));
Like was done in pull request #49
The text was updated successfully, but these errors were encountered: