[RTL/SV] Move UnpackedArrayType to SV dialect
- Dominant language
- C++
- Stars
- 2.2k
- Forks
- 524
- Avg merge
- 3d 2h
- Merged PRs (30d)
- 46
Description
`UnpackedArrayType` in the RTL dialect is more of a syntactic/behavioral construct that only makes sense in the SV world. We should move it over there.
Unfortunately, this is trickier than it sounds because the RTL dialect library currently does not depend on the SV dialect library (SV depends on RTL and we want lib dependencies to be a DAG) and there is various checking and other logic in RTL land that needs to be aware of this type.
That said, it is fixable and we should detangle this.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by locating UnpackedArrayType in the RTL dialect and the SV dialect library, then trace the RTL checking and related logic that recognizes the type. The work is complete when the type belongs to SV, the library dependency remains a DAG, and the affected RTL logic still handles it correctly.
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