google / google/xls

[enhancement] Put module_builder's functions in a shared package

Open
#1,643 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

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

Currently, module_builder puts functions in each module. With multi-proc codegen, this means you end up redefining a bunch of functions in many cases.

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

Accept the current behavior- it works.

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

At least for SystemVeriog, it might be nice to pull all these functions outside of their modules and put it in a package instead.

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.