This modifies the behavior of a stage to change the TargetDirectory to
a subdirectory derived from its application name before doing the work
of a Stage. This enables better structured output of multiple stages
where the outputs of each stage are placed in separate subdirectories.

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

This adds a new phase, TargetDirectoryManipulator, that enables
manipulation of the TargetDirectoryAnnotation using two annotations:

  - TargetDirectoryEnterAnnotation which changes the TargetDirectory
    to a subdirectory
  - TargetDirectoryExitAnnotation which enters the parent directory of
    the current TargetDirectory

This also adds a helper phase, TargetDirectoryExit, which will add a
TargetDirectoryExitAnnotation.

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

This moves the --no-dedup option to be FIRRTL-stage specific as
opposed to a global option common to all stages.

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

Remove scala-logging fully in favor of our own logger

There was some vestigial logging that conflicts with the homebrewed
logger used by most of the codebase

* Types containing bundles can't generally be converted to a single mask granularity

* Update ReplSeqMemTests to check for illegal masks

[Mergify] Automatically merge backport PRs when ready

* Closes #1242

Use Mergify to automate backporting to 1.2.x

getSimpleName considered harmful

This changes uses of `getSimpleName` to `getName`. The former throws
idiotic exceptions under Java 8, e.g., `getSimpleName` will fail if
used on a class inside an object.

This fixes a bug where any call to the `name` method of a custom
transform defined inside an object (or in an environment wrapping
things in objects like a REPL) will throw a malformed class name
exception. E.g., if you do this and run with `-ll info` or your custom
transform deletes annotations.

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

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

* Add spec for Analog type and attach statement

* Describe role of attaches in partial connection algorithm
* Change references that describe ground types where appropriate
* Closes #1194

* Fix typo

Add separate Issue and PR templates

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

Recursive literal lookup needs to be guarded against combinational loops
Added a test-case to illustrate the issue when CheckResets is run before CheckCombLoops

Supply a trait to allow user annotations to provide JsonProtocol type hints

* Closes #1162
* Instances of extmodules remain in the final hierarchy
* Extmodule definitions are not renamed or duplicated
* The rest of the pass may proceed as normal

* Fixes #1154
* Tests that #1154 example produces SyntaxErrorsException
* Generally helps catch trailing syntax errors
* Performance-neutral relative to previous grammar
* Recommended by antlr4 devs, can help performance in some cases
* See antlr/antlr4#1540

Guard initial blocks in emitted Verilog with `ifndef SYNTHESIS

Change findInstancesInHierarchy to return implicit top instance

Co-Authored-By: Jack Koenig <koenig@sifive.com>

* Change FIRRTL-internal API, affecting only one corner case
* Make API more "DWIM" and consistent with other methods
* Add test cases for findInstancesInHierarchy
* Update Scaladoc

Enhance CheckCombLoops errors with connection info