From 36227e961a1fa0626249d71edd8b6c829a79a2ee Mon Sep 17 00:00:00 2001 From: Jack Koenig Date: Tue, 12 May 2020 23:03:57 -0700 Subject: Remove expensive .distinct in Dedup --- src/main/scala/firrtl/transforms/Dedup.scala | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'src') diff --git a/src/main/scala/firrtl/transforms/Dedup.scala b/src/main/scala/firrtl/transforms/Dedup.scala index d7769820..f9e9db48 100644 --- a/src/main/scala/firrtl/transforms/Dedup.scala +++ b/src/main/scala/firrtl/transforms/Dedup.scala @@ -74,7 +74,11 @@ class DedupModules extends Transform with DependencyAPIMigration with PreservesA val dedupMap = DedupModules.deduplicate(c, noDedups.toSet, annos, renameMap) // Use old module list to preserve ordering - val dedupedModules = c.modules.map(m => dedupMap(m.name)).distinct + // Lookup what a module deduped to, if its a duplicate, remove it + val dedupedModules = c.modules.flatMap { m => + val mx = dedupMap(m.name) + if (mx.name == m.name) Some(mx) else None + } val cname = CircuitName(c.main) val map = dedupMap.map { case (from, to) => -- cgit v1.2.3