google / google/xls

[codegen] Opportunistically make names into the originals from the user input program

Open
#1,121 0 comments 0 reactions 0 assignees View on GitHub
codegen ir optimizer
Dominant language
C++
Stars
1.9k
Forks
283
Avg merge
2d 10h
Merged PRs (30d)
135

Description

Right now in the generated pipeline you'll see names like `p1_add_42_foo`, but arguably the `add_42` is only useful to compiler engineers and could be emitted as sideband metadata that we overlay on the verilog with our own bespoke debug tooling. It seems like we could opportunistically recover names. If we wanted to be conservative about the correspondence, we could try to tag when rewrites changed nodes semantically and avoid recovering the original names in those cases.

This was somewhat spurred by the nice better names in #1028 we could get opportunistically, there's something analogous we can do for codegen I bet.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.