google / google/xls

Factor out FIFO state in block evaluators

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

Description

In a current WiP change, I'm putting FIFO state in the register state map. This is convenient and avoids the need for API changes, but has some shortcomings.

Long term, it seems better to factor out FIFO state. Some benefits:
- Initializing registers is already clunky and the current approach makes it worse.
- Using std::deque or circular buffer is more performant.
- Factoring state out may make it easier to share FIFO implementation w/ the JIT.

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.