google / xls

XLS: Accelerated HW Synthesis

Home Page:http://google.github.io/xls/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Factor out FIFO state in block evaluators

grebe opened this issue · comments

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.