llvm / llvm/circt

Too many places memories are lowered in firrtl

Open
#4,694 0 comments 2 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
2.2k
Forks
524
Avg merge
3d 2h
Merged PRs (30d)
46

Description

as mentioned in #4687, memory lowering is spread through multiple passes in firrtl (seqmems, lowertypes, lowertohw, etc). We need to consolidate this concern to a pass.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reading issue #4687 and tracing memory lowering across the mentioned firrtl passes: seqmems, lowertypes, and lowertohw. Determine the intended single pass boundary and what each existing pass currently contributes. Done means the memory-lowering concern is consolidated into one pass without leaving duplicated lowering responsibilities across those passes.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
compilers
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.