chipsalliance / chipsalliance/rocket-chip
TraceSinkMonitor should be a SystemVerilog module
Open
- 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
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