google / google/xls

[enhancement] channel (and func input) flopping should be resolved in a codegen pass

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

Description

### What's hard to do? (limit 100 words)

Resolution of what type of flopping/buffering to add to ports is currently done in a somewhat ad-hoc manner in block_conversion.cc. This is difficult to test and complicated.

### Current best alternative workaround (limit 100 words)

N/A

### Your view of the "best case XLS enhancement" (limit 100 words)

This resolution should really be done as a resolution pass performed prior to block conversion which would make testing easier and keep everything much more self-contained. As codegen refactor continues this should be one of the things changed.

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.