google / google/xls

Codegen options should be per-proc

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

Description

With multi-proc codegen (#950) the current system applies the same codegen option to each proc, but you may want each proc to have its own settings (e.g. pipeline depth set per-proc, I/O flopping, etc.).

Some options might make sense to move out of flags into source. Some options should be moved into a map that has instance-path keys and options as values.

If we want to build options programmatically, we may want an options-proto-builder to help manage the complexity.

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.