google / google/xls

Block stitching instance numbers are fragile

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

Description

Block stitching adds instance number suffixes when instantiating blocks. New-style proc codegen might naturally clean this up, but in the interim it's annoying that instance names are fragile.

It would be better if the instance numbers were per-instantiated-block, e.g. instead of

```
inst A_0, inst B_1, inst A_2
```

you got

```
inst A_0
inst B_0
inst A_1
```

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.