sfcX/src/test/scala/firrtlTests/annotationTests, branch master Scala FIRRTL Compiler for chiselX fix tests that depend on Driver (#2429) 2021-11-23T18:56:03+00:00 Jiuyang Liu 2021-11-23T18:56:03+00:00 16714d6abeab5140e97d8370e8b901b4eb816fde Co-authored-by: sinofp <sinofp@tuta.io> 
Co-authored-by: sinofp <sinofp@tuta.io>
 Let firrtl based applications run despite loading unknown annotations (#2387) 2021-11-12T20:29:48+00:00 Chick Markley 2021-11-12T20:29:48+00:00 c6093cbcd4f2eb8acd44f3b9d4e7146448de172f An application like barstools may contain a main that loads an annotations file containing annotation classes that are not on it's classpath. This change allows unknown annotations to be preserved by wrapping them in a UnrecognizedAnnotation. If annotations are then output to a file, they will be unwrapped during serialization This feature can be enabled via an AllowUnrecognizedAnnotations annotation Co-authored-by: chick <chick.markley@sifive.com> Co-authored-by: Jack Koenig <koenig@sifive.com> 
An application like barstools may contain a main that loads an annotations file containing
annotation classes that are not on it's classpath. This change allows unknown annotations
to be preserved by wrapping them in a UnrecognizedAnnotation. If annotations are then output
to a file, they will be unwrapped during serialization
This feature can be enabled via an AllowUnrecognizedAnnotations annotation

Co-authored-by: chick <chick.markley@sifive.com>
Co-authored-by: Jack Koenig <koenig@sifive.com>
 Optimize Annotation.getTargets (#2244) 2021-05-21T22:42:53+00:00 Jack Koenig 2021-05-21T22:42:53+00:00 117b84a15a352451c1217155f96b09d098681baf 






Add JsonProtocol.serializeRecover (#2227)
2021-05-14T18:12:19+00:00

Jack Koenig

2021-05-14T18:12:19+00:00

c2d72fd8a4f3558094094eed2f4a12f85d080c41

This function will safely wrap any unserializeable annotations in
UnserializeableAnnotations so that they can be safely serialized to JSON
for logging.




This function will safely wrap any unserializeable annotations in
UnserializeableAnnotations so that they can be safely serialized to JSON
for logging.






Improve performance of LowerTypes renaming (#2024)
2020-12-16T00:41:53+00:00

Jack Koenig

2020-12-16T00:41:53+00:00

15013df6f6ac2dafeb35d7ed15cf95c7ac8a5bef

This is done by having LowerTypes uses two RenameMaps instead of one for
each module. There is one for renaming instance paths, and one for
renaming everything within modules.

Also add some utilities:
* TargetUtils for dealing with InstanceTargets
* RenameMap.fromInstanceRenames




This is done by having LowerTypes uses two RenameMaps instead of one for
each module. There is one for renaming instance paths, and one for
renaming everything within modules.

Also add some utilities:
* TargetUtils for dealing with InstanceTargets
* RenameMap.fromInstanceRenames






Change to Apache 2.0 License (#1901)
2020-09-17T01:52:16+00:00

Chick Markley

2020-09-17T01:52:16+00:00

ecb96e83324ea17cf38b7b90753d745d3c7f51bd












Don't use ResolvedAnnotationPaths in ConstProp nor DCE (#1896)
2020-09-15T02:43:01+00:00

Jack Koenig

2020-09-15T02:43:01+00:00

0c646459b52394e8a388143cee324b8af3dc7c09

Both use EliminateTargetPaths to duplicate modules based on annotations.
Currently, EliminateTargetPaths API is a little too limited so it
duplicates more than it should which effectively breaks Dedup whenever
DontTouchAnnotations are present.

Also, make ConstProp and DCE treat all HasDontTouches as local
annotations even if they are instance annotations. This is more
conservative but it is generally better to preserve deduplication than
to maximally optimize every instance.

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




Both use EliminateTargetPaths to duplicate modules based on annotations.
Currently, EliminateTargetPaths API is a little too limited so it
duplicates more than it should which effectively breaks Dedup whenever
DontTouchAnnotations are present.

Also, make ConstProp and DCE treat all HasDontTouches as local
annotations even if they are instance annotations. This is more
conservative but it is generally better to preserve deduplication than
to maximally optimize every instance.

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






Better error messages for unserializable annotations (#1885)
2020-09-05T00:20:01+00:00

Jack Koenig

2020-09-05T00:20:01+00:00

47c81ee3e68c96e94dafa389cb53b162e996a4df












All of src/ formatted with scalafmt
2020-08-15T02:47:53+00:00

chick

2020-08-15T02:47:53+00:00

6fc742bfaf5ee508a34189400a1a7dbffe3f1cac












Cleanup Named Targets (#1311)
2020-08-13T03:02:04+00:00

Schuyler Eldridge

2020-08-13T03:02:04+00:00

e1d3a0c64a48b4c9999ad59f15922db7c155c361

* Add CleanupNamedTargets Transform

Adds a new transform, CleanupNamedTargets, that will convert
ReferenceTargets to InstanceTargets if any ReferenceTargets are found
to be actually pointing at instances.

This fixes an issue where a ComponentName pointing at an instance will
be converted to a ReferenceTarget. Later on, this ReferenceTarget will
not be properly renamed by other transforms, e.g.,
EliminateTargetPaths.

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

* Update ScalaTests

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




* Add CleanupNamedTargets Transform

Adds a new transform, CleanupNamedTargets, that will convert
ReferenceTargets to InstanceTargets if any ReferenceTargets are found
to be actually pointing at instances.

This fixes an issue where a ComponentName pointing at an instance will
be converted to a ReferenceTarget. Later on, this ReferenceTarget will
not be properly renamed by other transforms, e.g.,
EliminateTargetPaths.

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

* Update ScalaTests

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