chipsalliance / chipsalliance/chisel

Support deduplication of modules where an output is being used as a source in BoringUtils.bore()

Open
#3,118 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
Scala
Stars
4.8k
Forks
658
Avg merge
18h 59m
Merged PRs (30d)
14

Description

@jackkoenig @seldridge

**Type of issue**: Feature Request

**Is your feature request related to a problem? Please describe.**

Sometimes it is useful to using BoringUtils.bore() with the source being an output of a deduplicated macro

**Describe the solution you'd like**

Deduplication should work.

**Describe alternatives you've considered**

Connecting to a wire in the module that instantiates the module that I want to bore to.

**Additional context**

Deduplication not working for Element:

https://scastie.scala-lang.org/9ILfIwIvQZa8EiIvL0lQzg

**What is the use case for implementing this feature?**

Contributor guide

Open the contributing guide

Research direction

Start by reproducing the linked Scastie example and reading BoringUtils.bore() behavior around outputs from deduplicated macros. Trace the deduplication handling for Element outputs and locate related tests. Done means BoringUtils.bore() supports a source that is an output of a deduplicated macro, with regression coverage.

Written by the indexing model from the issue text.

Assessment

Tech stack
scala
Domain
compilers
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.