chipsalliance/chisel

Add Bundle.Init (analogous to VecInit / Bundle.Lit)

Aperta

#2435 aperta il 7 mar 2022

Vedi su GitHubGuida contributor
 (6 commenti) (0 reazioni) (0 assegnatari)Scala (651 fork)batch import
good first issue

Metriche repository

Star
 (4658 stelle)
Metriche merge PR
 (Merge medio 3g 21h) (16 PR mergiate in 30 g)

Descrizione

I sketched out a version of this here: https://scastie.scala-lang.org/D0948BD1SgeZEqLGkVBoJg

Ideally this (and VecInit for that matter) would:

  1. compose
  2. utilize DataView -- (we probably cannot change VecInit but views here so that you can connect to the underlying fields would be nice)

Regardless, this sketch is useful on its own even without the more robust solution

Type of issue: bug report | feature request | documentation | other enhancement

Impact: API addition (no impact on existing code)

Development Phase: proposal

What is the use case for changing the behavior?

Useful API for users

Guida contributor

Direzione di ricerca
Esamina lo schizzo Scastie fornito (https://scastie.scala lang.org/D0948BD1SgeZEqLGkVBoJg) per comprendere l'API Bundle.Init proposta. Comprendi l'implementazione esistente di VecInit per garantire coerenza. Considera come Bundle.Init potrebbe comporsi con VecInit e utilizzare DataView per collegare i campi sottostanti. Implementa l'API e aggiungi test.
Tech stack
scala
Dominio
api
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
ScalaChisel
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70