llvm / llvm/circt

[FIRRTL] Move `lib/Firtool` and `lib/Reduce` under `lib/Tools`?

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

Description

`Firtool` living under `lib` looks a bit weird to me. MLIR uses `lib/Tools` as a place to put tool-specific libraries so I think we should move them to `lib/Tools`.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by inspecting lib/Firtool and lib/Reduce, then search the repository for references to both paths and the expected lib/Tools layout. Check the related build configuration and any affected tests or entry points; done means the libraries are under lib/Tools and all references and builds still work.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
build-system, tooling
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.