Merge the headers in `include/circt/Conversion`?
- Dominant language
- C++
- Stars
- 2.2k
- Forks
- 524
- Avg merge
- 3d 2h
- Merged PRs (30d)
- 46
Description
We have a number of header files (one per library) in `include/circt/Conversion/*.h` which generally expose one or two functions.
At the same time, we have a single `include/circt/Conversion/Passes.h` file which has all the passes for all the libraries.
I think we should merge all the other things into a single `include/circt/Conversion/Conversions.h` header. It will be simpler, and doesn't change the layering.
Contributor guide
No contributing guide indexed for this repository
Research direction
Review the header files under include/circt/Conversion/*.h alongside include/circt/Conversion/Passes.h. Identify the conversion declarations that should move into a single include/circt/Conversion/Conversions.h while preserving the existing layering. Done means the conversion interfaces are consolidated as proposed without changing that layering.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- compilers
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100