From 02e46bdb40b76c9f7803dd1ae4f18b388f9d55a4 Mon Sep 17 00:00:00 2001 From: Jack Koenig Date: Fri, 17 Dec 2021 09:25:38 -0800 Subject: Modify and optimize performance of propagate annotations (#2393) * Change AnnotationSeq underlying from List to Seq It was nothing but pointless copying. * Make propagateAnnotations faster There was lots of expensive logic for very little benefit.--- src/test/scala/firrtlTests/AnnotationTests.scala | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'src/test') diff --git a/src/test/scala/firrtlTests/AnnotationTests.scala b/src/test/scala/firrtlTests/AnnotationTests.scala index 7837b4ba..a51a8e77 100644 --- a/src/test/scala/firrtlTests/AnnotationTests.scala +++ b/src/test/scala/firrtlTests/AnnotationTests.scala @@ -45,7 +45,9 @@ abstract class AnnotationTests extends LowFirrtlTransformSpec with Matchers with r.annotations.toSeq should contain(ta) } - "Deleting annotations" should "create a DeletedAnnotation" in { + // This test is no longer true as of 1.5.0-RC2, see + // https://github.com/chipsalliance/firrtl/pull/2393 + "Deleting annotations" should "create a DeletedAnnotation" ignore { val transform = Dependency[DeletingTransform] val compiler = makeVerilogCompiler(Seq(transform)) val input = -- cgit v1.2.3