Refactor FirMemory / analyzeMemOp, etc to reduce duplication
Open
- Dominant language
- C++
- Stars
- 2.2k
- Forks
- 524
- Avg merge
- 3d 2h
- Merged PRs (30d)
- 46
Description
There is a bunch of copy and pasted code between LowerToRTL and RTLMemSimImpl, these need to get merged into the FIRRTL dialect as helpers.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by comparing the duplicated code in LowerToRTL and RTLMemSimImpl, focusing on FirMemory and analyzeMemOp. Identify the shared operations that belong as helpers in the FIRRTL dialect. Done means the duplication is removed and both callers use the consolidated helpers.
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
- Needs clarification
- Newbie friendliness
- 35/100