Skip to content

Source debug_05 - #707

Open
desmonddak wants to merge 35 commits into
intel:mainfrom
desmonddak:source_debug
Open

desmonddak wants to merge 35 commits into
intel:mainfrom
desmonddak:source_debug

Conversation

@desmonddak

Copy link
Copy Markdown
Contributor

Description & Motivation

This branch adds an FLC (File,Line,Column) format output and cross-probing capabilities for ROHD when it creates a netlist.
These entries can help trace signals and cells back to the original ROHD format or if other outputs are created to the relevant
line numbers for those outputs based on the place where those signals are named(). It is biased toward listing assignments before definitions.

Related Issue(s)

None.

Testing

Tests are included for simple cross-probing

Backwards-compatibility

Is this a breaking change that will not be backwards-compatible? If yes, how so?
No

Documentation

Does the change require any updates to documentation? If so, where? Are they included?

Yes, we added a cross_probing.md among other source documentation.

@desmonddak desmonddak changed the title Source debug Source debug_05 Sep 15, 2026
@desmonddak
desmonddak force-pushed the source_debug branch 10 times, most recently from 275a0ac to 0f0213b Compare September 19, 2026 06:21
@desmonddak
desmonddak force-pushed the source_debug branch 3 times, most recently from 477f307 to c145c10 Compare October 8, 2026 23:57
desmonddak and others added 16 commits October 8, 2026 17:30
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Remove the obsolete material_ui upper bound and cupertino_ui override.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This reverts commit 4842ff5101850584f6302a458fd29a9c566f60f8.
Signed-off-by: Desmond A. Kirkpatrick <desmond.a.kirkpatrick@intel.com>
Signed-off-by: Desmond A. Kirkpatrick <desmond.a.kirkpatrick@intel.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Desmond A. Kirkpatrick <desmond.a.kirkpatrick@intel.com>
desmonddak and others added 19 commits October 8, 2026 17:30
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The packed-output split pass runs after naming and rebinds a submodule
output port to an unnamed SynthLogicConcat. SystemC binds ports by
signal name and cannot bind an output to a concatenation, so reading
the name asserted ("Name has not been picked ... [_s]").

Add a supportsPackedOutputPortConcatenation hook (default true) that
gates the split pass, and disable it for SystemC so the intermediate
bus is retained and named. Also give packed bit/range references a
nameOrNull so consumers can query names safely.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Desmond A. Kirkpatrick <desmond.a.kirkpatrick@intel.com>
Signed-off-by: Desmond A. Kirkpatrick <desmond.a.kirkpatrick@intel.com>
Signed-off-by: Desmond A. Kirkpatrick <desmond.a.kirkpatrick@intel.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
When a packed submodule output is split, its output mapping becomes an
unnamed SynthLogicConcat. _buildSvLineMap read .name on that mapping
and asserted. Expand concat parts and use nameOrNull so each named
part receives an instance line-map entry.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant