google / google/xls

ram_rewrite_pass calls UniquifyNodeName() too much

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

Description

This is most visible with port names- if you request the same ram name, you get `port_name`, `port_name__3`, `port_name__6`, etc. because `UniquifyNodeName()` is being called too much. This also happens to internal signals.

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.