[FIRRTL] Support read-under-write modifiers on CHIRRTL mem constructs
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 2.2k
- Forks
- 524
- Avg merge
- 3d 2h
- Merged PRs (30d)
- 46
Description
CHIRRTL memories were supposed to have an undefined, old, or new read-under-write modifiers after the memory declaration. E.g.:
smem mem1 : UInt<8> [8] old
smem mem2 : UInt<8> [8] new
smem mem3 : UInt<8> [8] undefined
This then should be copied over to the FIRRTL memory that is inferred.
Presently, this modifier was never serialized in the textual format coming out of Chisel. (It was just dropped.)
This will change in the actual release of Chisel 3.6.
Contributor guide
No contributing guide indexed for this repository
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.
Research direction
Locate the CHIRRTL memory parsing and textual serialization entry points, then trace how inferred FIRRTL memories are produced. Verify the undefined, old, and new modifiers are preserved from declarations through inferred memories, and run the relevant memory-format tests if available.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- compilers
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100