llvm / llvm/circt

--top isn't respected in driver.cpp for circt-rtl-sim.py

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

Description

driver.cpp includes Vtop.h, which assumes the toplevel module is named top. circt-rtl-sim.py takes a parameter to set the toplevel name to something else, but the driver still assumes it's "top".

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by tracing how circt-rtl-sim.py passes the --top parameter into driver.cpp, then inspect the Vtop.h include and related generated-module assumptions. Done means the simulator uses a non-top toplevel name successfully instead of always requiring a module named top.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp, python
Domain
cli, tooling
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 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.