chipsalliance / chipsalliance/Surelog
Remove variables::VpiSigned?
Open
- Dominant language
- C++
- Stars
- 475
- Forks
- 90
- Avg merge
- 1h 39m
- Merged PRs (30d)
- 37
Description
`variables` have `typespec` and `typespec` also have `VpiSigned`. Is `variables::VpiSigned` remnant from time before `typespec`? Can this be removed?
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by locating the declarations and usages of variables::VpiSigned and typespec::VpiSigned, then inspect any tests or API consumers that reference them. Determine whether the variable member is redundant and verify that removing or retaining it preserves the relevant behavior and tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- compilers
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100