The-OpenROAD-Project / The-OpenROAD-Project/OpenROAD
ram: add placement sanity checks to the regression tests (`check_placement`, etc.)
Open
@rovinski is already working on this.
Since May 30, 2026.
ram
- Dominant language
- Verilog
- Stars
- 3.1k
- Forks
- 1k
- Avg merge
- 2d 23h
- Merged PRs (30d)
- 136
Description
The ram tests don't feature any basic sanity tests to ensure that the result is correct. Sometimes visual inspection isn't sufficient.
- Tests should add
check_placementafter running in order to ensure that cells were not placed on top of each other or are otherwise overlapping - The power grid should be checked to ensure that there are no unconnected pins
- I think that routing violations are already checked, so that's ok
- Formal equivalence should be checked, but that can be a separate issue
- Any other built-in checks that can be run
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.