Skip to content

Commit

Permalink
Avoid wide signals in sensitivity lists of immediate assertions
Browse files Browse the repository at this point in the history
Verilator has decent support for various types of assertions by now but
dislikes wide signals (e.g., wider than 64 bits) in sensitivity lists.
  • Loading branch information
michael-platzer committed Oct 1, 2024
1 parent 10dac0f commit 38a94fe
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/addr_decode_dync.sv
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,7 @@ module addr_decode_dync #(
// check_start: Enforces a smaller start than end address.
// check_idx: Enforces a valid index in the rule.
// check_overlap: Warns if there are overlapping address regions.
always @(addr_map_i or config_ongoing_i) #0 begin : proc_check_addr_map
always_comb begin : proc_check_addr_map
if (!$isunknown(addr_map_i) && ~config_ongoing_i) begin
for (int unsigned i = 0; i < NoRules; i++) begin
check_start : assume (Napot || addr_map_i[i].start_addr < addr_map_i[i].end_addr ||
Expand Down
2 changes: 1 addition & 1 deletion src/multiaddr_decode.sv
Original file line number Diff line number Diff line change
Expand Up @@ -133,7 +133,7 @@ module multiaddr_decode #(

// These following assumptions check the validity of the address map.
// check_idx: Enforces a valid index in the rule.
always @(addr_map_i) #0 begin : proc_check_addr_map
always_comb begin : proc_check_addr_map
if (!$isunknown(addr_map_i)) begin
for (int unsigned i = 0; i < NoRules; i++) begin
// check the SLV ids
Expand Down

0 comments on commit 38a94fe

Please sign in to comment.