google / google/xls

verilog bazel rules should emit System/VerilogInfo

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

Description

It seems that currently `dslx/ir_verilog` rules are connected to `bazel_rules_hdl`'s `verilog_library` thru brick and mortar files, see:
https://github.com/google/xls/blob/75930f7667fdc725d987bd6d02b8719b35c9f6dd/xls/examples/BUILD#L407-L412

It'd be nice to leverage instead structured bazel types like `VerilogInfo`: https://github.com/hdl/bazel_rules_hdl/blob/main/verilog/providers.bzl#L19, it could also allow simulation rules to ask for specific variant of (system)verilog depending of their frontend capabilities.

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.