# Conflicts:
#	src/main/scala/firrtl/annotations/JsonProtocol.scala
#	src/main/scala/firrtl/transforms/Dedup.scala
#	src/test/scala/firrtlTests/FirrtlSpec.scala
#	src/test/scala/firrtlTests/FlattenTests.scala
#	src/test/scala/firrtlTests/InlineInstancesTests.scala
#	src/test/scala/firrtlTests/PassTests.scala
#	src/test/scala/firrtlTests/transforms/TopWiringTest.scala

* Provide an annotation mix-in that marks RTs as dontTouch

* Update src/main/scala/firrtl/transforms/OptimizationAnnotations.scala

Co-Authored-By: Albert Magyar <albert.magyar@gmail.com>

* Update src/test/scala/firrtlTests/DCETests.scala

Co-Authored-By: Albert Magyar <albert.magyar@gmail.com>

* Update src/main/scala/firrtl/transforms/OptimizationAnnotations.scala

* Update OptimizationAnnotations.scala

Co-authored-by: Albert Magyar <albert.magyar@gmail.com>

Make mergify open backport PRs & signal on failed cherry-picks

* Add firrtl-json serializers

* Added support for ports, info. Added docs and tests

* Avoid IndexOutOfBoundsException when Bits has too few consts

* Check for negative consts in all relevant primops

* Use BigInt for all checks on primop constants

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

Revert inline nots

* Feedback from @jackkoening
* Merge into same stage as Ops to avoid Travis delays

This reverts commit f77487d3.

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

* Closes #890

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

EliminateTargetPaths and Unreachable Modules

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

Adds a case to CircuitState.resolvePaths such that if no targets are
requested, then no ResolvePaths annotations are added.

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

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

Omnibus Dependency API Updates

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

This adds a prettyPrint method to the DependencyManager to enable
textual visualization of the TransformLikes that a DependencyManager
determines need to be run.

This also cleans up the GraphViz visualization with better edge
coloring and now uses the `name` method when labeling graphviz nodes.

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

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

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

This makes a change to the Dependency API that breaks chisel3. This
needs to [skip chisel tests], but is fixed with
https://github.com/freechipsproject/chisel3/pull/1270

.

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

Add more docs / tests for DiGraph reachableFrom method

This reverts commit eabc3855.

* Revert "Infer resets last connect semantics (#1291)"
* Fix handling of invalidated and undriven components of type Reset
* Run CheckTypes after InferResets
* Make reset inference bidirectional on connect
* Support AsyncResetType in RemoveValidIf
* Fix InferResets for parent constraints on child ports
* Apply suggestions from code review
* Add ScalaDoc to InferResets

Co-authored-by: Albert Magyar <albert.magyar@gmail.com>
Co-authored-by: Schuyler Eldridge <schuyler.eldridge@gmail.com>

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

Constant prop binary PrimOps with matching arguments

* Send in the Yosys

* Add SimplifyBinaryOp trait
* Add extra functionality to comparison folding
* Add tests
* Fix comments from review

Add InstaceGraph (Un)?Reachable Helpers