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/main/scala/firrtl/transforms/DedupAnnotations.scala | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/main/scala/firrtl/transforms') diff --git a/src/main/scala/firrtl/transforms/DedupAnnotations.scala b/src/main/scala/firrtl/transforms/DedupAnnotations.scala index cad4d2be..9aad2fee 100644 --- a/src/main/scala/firrtl/transforms/DedupAnnotations.scala +++ b/src/main/scala/firrtl/transforms/DedupAnnotations.scala @@ -95,7 +95,7 @@ class DedupAnnotationsTransform extends Transform with DependencyAPIMigration { def execute(state: CircuitState): CircuitState = CircuitState( state.circuit, state.form, - DedupAnnotationsTransform.dedupAnnotations(state.annotations.underlying, InstanceKeyGraph(state.circuit)), + DedupAnnotationsTransform.dedupAnnotations(state.annotations.toSeq, InstanceKeyGraph(state.circuit)), state.renames ) } -- cgit v1.2.3