This moves name sanitation a little earlier and ensures that the
sanitized names are use more consistently throughout Chisel internals.

This fixes a bug in D/I where unsanitary names would lead to a crash.

(cherry picked from commit 5c282db5)

* SRAM API: Add multiple-clocked port API (#3383)

This PR adds new APIs for the SRAM module to instantiate a number of ports connected to individual clocks. This allows the creation of memories driven by multiple clocks, for use cases like clock domain crossover.

(cherry picked from commit d79ae71e

)

---------

Co-authored-by: Jared Barocsi <82000041+jared-barocsi@users.noreply.github.com>
Co-authored-by: Jared Barocsi <jared.barocsi@sifive.com>

[svsim] Don't redirect simulation stderr to a pipe (#3403)

(cherry picked from commit e51fa08c

)

Co-authored-by: George Lyon <GeorgeLyon@users.noreply.github.com>

* Deprecate calling .viewAs on non-hardware (#3395)

It's an error in most cases already, and in the cases it isn't, make it
a warning. Also fix a crash that could occur when viewing a non-Data
containing non-hardware Data.

(cherry picked from commit 1e524de9

)

# Conflicts:
#	src/test/scala/chiselTests/experimental/DataView.scala

* Resolve backport conflicts

---------

Co-authored-by: Jack Koenig <koenig@sifive.com>

This is not a functional bug. The recursive form of reify for Element
would use the .topBinding from the wrong argument, so it would recurse
an extra time, but it would ultimately converge to the right value.

(cherry picked from commit 374b45b9

)

Co-authored-by: Jack Koenig <koenig@sifive.com>

* Support literals and DontCare in DataView targets (#3389)

(cherry picked from commit 0f803cd8

)

# Conflicts:
#	src/test/scala/chiselTests/experimental/DataView.scala

* Resolve backport conflicts

---------

Co-authored-by: Jack Koenig <koenig@sifive.com>

* log2 functions
* unsignedBitLength

(cherry picked from commit 88fecfc1

)

Co-authored-by: Jack Koenig <koenig@sifive.com>

(cherry picked from commit 2e95804f

)

Co-authored-by: Jared Barocsi <82000041+jared-barocsi@users.noreply.github.com>

Co-authored-by: Jiuyang Liu <liu@jiuyang.me>
Co-authored-by: Jack Koenig <koenig@sifive.com>
(cherry picked from commit 8851faed

)

Co-authored-by: Jared Barocsi <82000041+jared-barocsi@users.noreply.github.com>

(cherry picked from commit b910bf94

)

Co-authored-by: Jack Koenig <koenig@sifive.com>

* Allow IntrinsicModule to be used in Instantiate

Change a few `RawModule`s to `BaseModule`s to allow for intrinsic
modules to be used with the instantiation API.

* Add tests for Instantiate of non-RawModules

---------

Co-authored-by: Fabian Schuiki <fabian@schuiki.ch>
(cherry picked from commit 951f8b7c

)

Co-authored-by: Jack Koenig <koenig@sifive.com>

(cherry picked from commit e718aac1

)

Co-authored-by: Megan Wachs <megan@sifive.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

(cherry picked from commit c81c9f18

)

Co-authored-by: Megan Wachs <megan@sifive.com>

(cherry picked from commit b83f3695

)

Co-authored-by: Jack Koenig <koenig@sifive.com>

(cherry picked from commit 308eb6a4

)

Co-authored-by: Jack Koenig <koenig@sifive.com>

(cherry picked from commit 6e8eee88

)

Co-authored-by: Jack Koenig <koenig@sifive.com>

This enables Chisel users to use these newer Scala versions.

(cherry picked from commit b22c3ba2

)

Co-authored-by: Jack Koenig <koenig@sifive.com>

(cherry picked from commit e20a0380

)

Co-authored-by: Jack Koenig <koenig@sifive.com>

* Make svsim shutdown gracefully when the test throws an exception (#3309)

(cherry picked from commit 36a2ddff

)

* Waive false binary compatibility failure

The constructor is package private so user code cannot observe this
change.

---------

Co-authored-by: George Lyon <GeorgeLyon@users.noreply.github.com>
Co-authored-by: Jack Koenig <koenig@sifive.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

(cherry picked from commit 9e392013

)

Co-authored-by: Jared Barocsi <82000041+jared-barocsi@users.noreply.github.com>

(cherry picked from commit 1697f39f

)

Co-authored-by: Daniel Resnick <danielzresnick@gmail.com>

The updated versions fix warnings about deprecated actions.

(cherry picked from commit 21d8f6b7

)

Co-authored-by: Jack Koenig <koenig@sifive.com>

Co-authored-by: jackkoenig <jackkoenig@users.noreply.github.com>

* Optimize BitPat equals, overlap, and cover (#3285)

(cherry picked from commit f590ac9f

)

* Fix binary compatibility issue

---------

Co-authored-by: Jack Koenig <koenig@sifive.com>

* Add Top-level parameterized reset type
* Add the ability to set the top reset type for any Module

(cherry picked from commit 5406b4d5

)

Co-authored-by: Megan Wachs <megan@sifive.com>

This does not seem to be required by Scala, but it is required by newer
versions of SemanticDB (and thus to use Scala Metals).

(cherry picked from commit 08369ef8

)

Co-authored-by: Jack Koenig <koenig@sifive.com>

* Enable .viewAsSupertype to work on Records (#3267)

- Enable .viewAsSupertype to work on Records
- fix a nondeterminism error in DataView noticeable on error reporting

(cherry picked from commit bdb04ad7

)

* Waive binary compatibility changes

---------

Co-authored-by: Megan Wachs <megan@sifive.com>
Co-authored-by: Jack Koenig <koenig@sifive.com>

(cherry picked from commit b542b6ab

)

Co-authored-by: Sihao Liu <sihao@cs.ucla.edu>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Change what DataView allows in its type checking to enable Reset to be
viewed as either a UInt<1> or an AsyncReset.  This enables more flexible
DataViews which match already allowable Chisel and FIRRTL connection
semantics.

Signed-off-by: Schuyler Eldridge <schuyler.eldridge@sifive.com>
(cherry picked from commit 44ce96cf

)

Co-authored-by: Schuyler Eldridge <schuyler.eldridge@sifive.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

(cherry picked from commit 86ec9328

)

Co-authored-by: Jack Koenig <koenig@sifive.com>

* Add new workflow to fixup backports for release notes (#3252)

The Mergify-generated backports do not currently include the information
needed by the release notes generation automation. This new workflow
will copy such information over to backport PRs.

(cherry picked from commit 86ccd1b2)

* Fix backport-fixup for labels with spaces (#3255)

(cherry picked from commit 8f7e7e68

)

---------

Co-authored-by: Jack Koenig <koenig@sifive.com>

(cherry picked from commit 7b57e591

)

Co-authored-by: Jack Koenig <koenig@sifive.com>

Co-authored-by: jackkoenig <jackkoenig@users.noreply.github.com>

(cherry picked from commit bc4d8f9a

)

Co-authored-by: Jack Koenig <koenig@sifive.com>

Undo removal of address intermediate wire in `SyncReadMem.read`, `SyncReadMem.readWrite` (#3240) (#3241)

(cherry picked from commit c10d6494

)

Co-authored-by: Jared Barocsi <82000041+jared-barocsi@users.noreply.github.com>

* Mark generated wires for read and readWrite as temporary
* Remove semicolons
* Remove intermediate wire for read/readWrite port address
* Convert usages of UnlocatableSourceInfo into macro applications

(cherry picked from commit e078c0f2

)

Co-authored-by: Jared Barocsi <82000041+jared-barocsi@users.noreply.github.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* Add .unsafe to connectables

* Added .as and .squeezeAllAs to connectable

* Make connectable exceptions error at end of elaboration

(cherry picked from commit 4a5cbfd0

)

Co-authored-by: Adam Izraelevitz <adam.izraelevitz@sifive.com>

Bump firtool version to get parsing of new connect/invalidate
statements.

Signed-off-by: Schuyler Eldridge <schuyler.eldridge@sifive.com>
(cherry picked from commit f9f77882

)

Co-authored-by: Schuyler Eldridge <schuyler.eldridge@sifive.com>

(cherry picked from commit 79d2f8f1

)

Co-authored-by: Jack Koenig <koenig@sifive.com>

(cherry picked from commit 8ca8b2e1

)

Co-authored-by: Jack Koenig <koenig@sifive.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>