llvm / llvm/circt

[HW] HWModule (et. al.) should have separate symbol and name to be emitted

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

Description

Given all the talk about the mistake of conflating symbols with names, I had assumed that this had been fixed in the HW dialect. Is there some reason it hasn't? Or is this just waiting on someone to do it?

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reading the HW dialect implementation and the surrounding discussion about conflating symbols with names. Determine how HWModule and related entities currently represent and emit these concepts. Done means the required separation is designed and implemented consistently, with agreement on the intended behavior.

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
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.