chipsalliance / chipsalliance/rocket-chip

TraceSinkMonitor should be a SystemVerilog module

Open
#3,744 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Scala
Stars
3.9k
Forks
1.3k
Avg merge
5d 13m
Merged PRs (30d)
1

Description

The `TraceSinkMonitor` is a blackbox module wrapping around `/vsrc/TraceSinkMonitor.v`. However, inside the file, it is using the `final` block construct, a SystemVerilog feature.

The file extension should be changed to `.sv` to inform the simulation/synthesis tool that this is a SystemVerilog module, not a Verilog one.

Contributor guide

Open the contributing guide

Research direction

Start with /vsrc/TraceSinkMonitor.v and inspect how the TraceSinkMonitor blackbox is referenced by the project. Rename the file to use the .sv extension, then verify that the simulation or synthesis flow still finds and accepts the module.

Written by the indexing model from the issue text.

Assessment

Domain
embedded-iot
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.