lowRISC / lowRISC/opentitan

[bazel] Eliminate the use of `new_rules` in dvsim configs

Open
#19,973 0 comments 0 reactions 0 assignees View on GitHub
SW:Build System Tool:dvsim
Dominant language
SystemVerilog
Stars
3.6k
Forks
1.1k
Avg merge
2d 22h
Merged PRs (30d)
141

Description

We are using the flag `new_rules` in dvsim `hjson` files to indicate to DV test benches that certain test programs are being built with the new bazel rules and do not use the `opentitan_functest` naming conventions.

After all of the rules are converted, the older naming convention will be obsolete and the `new_rules` flag can be eliminated.

Contributor guide

Open the contributing guide

Research direction

Search the dvsim HJSON configurations for the `new_rules` flag and inspect the related Bazel rule conversion state. Confirm that the older `opentitan_functest` naming convention is no longer needed, then remove the flag and verify that DV test benches still build and run without it.

Written by the indexing model from the issue text.

Assessment

Domain
build-system, testing
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.