* Update reported width from div/rem to match FIRRTL results

* Add tests for width of % and / on UInt and SInt

* Add loop-based test for known UInt/SInt op result widths

Co-authored-by: Jack Koenig <koenig@sifive.com>
(cherry picked from commit 98ce9194

)

Co-authored-by: Albert Magyar <albert.magyar@gmail.com>
Co-authored-by: Jack Koenig <koenig@sifive.com>

* Bump to Scala 2.12.13 (#1766)

(cherry picked from commit 445b5cec)

* Fix chisel3-plugin binary compatibility checking

chisel3-plugin not yet published for Scala 2.12.13, check binary
compatibility with Scala 2.12.12 instead.

If a method passed to higher function does not return any value,
it is prefer to use `foreach` instead of `map`.

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
(cherry picked from commit 654db98e

)

Co-authored-by: SoyaOhnishi <ohnishi.soya@arc.cs.okayama-u.ac.jp>

(cherry picked from commit d0db0b8b

)

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

This is a nit fix. no logic is changed.

* Rename `typeName` to `enumTypeName` in ScalaDoc
* Add return type at public method
* Rename `enum_records` into `enumRecords` to retain name consistency

Co-authored-by: Jack Koenig <koenig@sifive.com>
(cherry picked from commit 2a967670

)

Co-authored-by: SoyaOhnishi <ohnishi.soya@arc.cs.okayama-u.ac.jp>

(cherry picked from commit e63b9667

)

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

This is useful for libraries to guard operations implemented via
annotations or BlackBoxes by the current when predicate

(cherry picked from commit cdb7bb27

)

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

Make `toTarget` fail if called on a Literal (or would otherwise not serialize properly) (#1714) (#1721)

* Add (failing) Test for Data toTarget calls

Add scaladoc and clean up test

* Builder: don't let .toTarget pass if it won't be able to deserialize properly later

* Update src/test/scala/chiselTests/ReferenceTargetSpec.scala

* Rename and simplify tests for literal toTarget

(cherry picked from commit b578aa5e

)

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

(cherry picked from commit a600a95b

)

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

* Remove CircleCI (#1702)

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
(cherry picked from commit eae69e95

)

# Conflicts:
#	.circleci/config.yml

* Resolve merge conflicts

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

* Switch to using Github Actions CI (#1690)

MiMa binary compatibility checks are now run on master, but the
mimaPreviousArtifacts are empty which makes the check a no-op. This
helps keep both the build.sbt and CI more consistent between master and
stable branches.

.mergify.yml is also updated with mergify.sc from chisel-repo-tools
using the following configuration:

conditions:
  - status-success=all tests passed
branches:
  - 3.2.x
  - 3.3.x
  - 3.4.x
(cherry picked from commit 25a417f8

)

# Conflicts:
#	.mergify.yml
#	build.sbt
#	project/plugins.sbt

* Resolve merge conflicts

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

- fix build.sbt
- fix build.sc

(cherry picked from commit 315ff8c6

)

Co-authored-by: Chick Markley <chick@qrhino.com>

* Builder: use LazyLogging.logger.warn to print elaboration message

* add deprecation for chisel3.internal.ErrorLog.info.

* add test to check elaboration message still exist.

Co-authored-by: Kevin Laeufer <kevin.laeufer@sifive.com>
(cherry picked from commit e8f90ef5

)

Co-authored-by: Jiuyang Liu <liu@jiuyang.me>

(cherry picked from commit ba05dcaf

)

Co-authored-by: Kevin Laeufer <electron.kiwi@gmail.com>

(cherry picked from commit e11d8b85

)

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

(cherry picked from commit e6192ea7

)

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

* Refine autonaming to have more intuitive behavior (#1660)

* Refine autonaming to have more intuitive behavior

Last name in an Expression wins, while the first Statement to name wins.
This is done via checking the _id of HasIds during autonaming and only
applying a name if the HasId was created in the scope of autonaming.
There is no change to .autoSeed or .suggestName behavior.

Behavior of chisel3-plugins from before this change is maintained.

* Update docs with naming plugin changes

(cherry picked from commit 1260f7c8

)

* Waive binary incompatibility false positive in 2.11

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

The SecurityManager is global so is not thread-safe. This is the source
of flaky tests in FIRRTL CI.

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
(cherry picked from commit 9f1d6cbb

)

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

This is a refinement of the assertion added in #1616 then removed in
 #1654. Because Records now set the refs of children upon binding,
later, unbound Records could incorrectly override the refs. The first
set should win.

(cherry picked from commit 8187318e

)

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

* Remove Data.setRef assertion (#1654)

It causes issues for some legal (if awkward) patterns. A larger refactor
of when refs are set could reinstate this check.

(cherry picked from commit 679dd54a

)

# Conflicts:
#	core/src/main/scala/chisel3/internal/Builder.scala

* Fix merge conflict

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

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
(cherry picked from commit 70538487

)

Co-authored-by: John Ingalls <43973001+ingallsj@users.noreply.github.com>

* Bugfix - adding external modules was broken (#1649)


(cherry picked from commit d21fe713

)

* Remove test because not backportable

* Remove extra space

Co-authored-by: Adam Izraelevitz <azidar@gmail.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* ExtModule's lacked support built in support for providing
the verilog source. This changes creates traits that
can be used with ExtModule to provide the support currently found in
BlackBox

- Add support for ExtModule helpers
  - HasExtModuleResource to use addResource
  - HasExtModuleInline to use setInline
  - HasExtModulePath to use addPath
- Add tests of the above support.
  - Note: These tests use Stage instead of Driver
- Added ScalaDoc for HasBlackBoxInline#setInline

* Fix the danged trailing commas.

* Change to use `.transform` as the correct API for `ChiselStage`

(cherry picked from commit fd92809e

)

Co-authored-by: Chick Markley <chick@qrhino.com>

This requires a combination of things, but it happens to be a
combination used by Diplomacy in Rocket Chip. It must be a Record in
compatibility code with Vecs as fields and a mix of components with and
without set directions.

(cherry picked from commit 0979133b

)

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

(cherry picked from commit 362a3e35

)

Co-authored-by: Adam Izraelevitz <azidar@gmail.com>

* Fix crosslinks in mdoc. Can't use md suffix

* Removed all .md crossrefs

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
(cherry picked from commit 58fa3c11

)

Co-authored-by: Adam Izraelevitz <azidar@gmail.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* Use Data refs for name prefixing with aggregate elements (#1616)

* Use Data refs for name prefixing with aggregate elements

Vecs set the refs of their elements upon construction of those elements.
In the past, Records haven't set their elements refs until module close,
but it can be done sooner. Doing it upon binding means that refs will at
least be available for Records used in hardware elements. Since only
bound Data can be connected to anyway, Aggregate elements being
connected to will always have a ref which we can then use for creating
naming prefixes.

* Add tighter correctness checks

* Handle more cases in connection prefixing

Add support for forcing setRef to override a previous setting. This
is only used by BlackBox ports which need to drop their io prefix.

Also add a Try() around Data.bindingToString which sometimes throws
exceptions when being used to .toString a Data in an error message.

* Strip trailing spaces in names in compiler plugin

(cherry picked from commit 0745dede

)

* Resolve binary compatibility issues

Change to Builder.scala is cleaner than waiving all of the (safe to
waive) failures.

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

* Bugfix - module name collision for injecting aspect (#1635)

* Bugfix - module name collision for injecting aspect

* Fixed mechanism to avoid module name collisions

* Added comments for reviewer feedback

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
(cherry picked from commit 1b6bd89d

)

* Update InjectingAspect.scala

Add additional function for backwards compatibility.

Co-authored-by: Adam Izraelevitz <azidar@gmail.com>

* Added forcename transform and tests

* Added documentation and additional error checking

* Added mdoc. Added RunFirrtlTransform trait

* Removed TODO comment

* Addressed reviewer feedback

* Removed trailing comma

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
(cherry picked from commit 2d98132d

)

Co-authored-by: Adam Izraelevitz <azidar@gmail.com>

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 d5db3881

)

Co-authored-by: PENGUINLIONG <admin@penguinliong.moe>

Causes issues when building the website, using these docs.

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
(cherry picked from commit 55809b1d

)

Co-authored-by: Adam Izraelevitz <azidar@gmail.com>

* Fix `-e` option causing ChiselStage.emit* to error

Signed-off-by: Schuyler Eldridge <schuyler.eldridge@sifive.com>

* Add test of `-e` ChiselStage behavior

Signed-off-by: Schuyler Eldridge <schuyler.eldridge@sifive.com>

* fixup! Add test of `-e` ChiselStage behavior

(cherry picked from commit 26deb770

)

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