Skip to content

Commit

Permalink
Change InlineSpec to expect "_" and not "$"
Browse files Browse the repository at this point in the history
A FIRRTL change to inlining changes the inlined instance delimiter to "_"
from "$". This change is reflected here in an update to the associated
Chisel tests for the Inline API.

Signed-off-by: Schuyler Eldridge <[email protected]>
  • Loading branch information
seldridge committed Oct 5, 2018
1 parent dafdeab commit 1f24aa0
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/test/scala/chiselTests/InlineSpec.scala
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ class InlineSpec extends FreeSpec with ChiselRunners with Matchers {
}
"low FIRRTL should contain only instance z" in {
val instances = collectInstances(firrtlResult.circuitState.circuit, Some("Top")).toSet
Set("Top", "Top.x$sub", "Top.y$sub", "Top.z", "Top.z.sub") should be (instances)
Set("Top", "Top.x_sub", "Top.y_sub", "Top.z", "Top.z.sub") should be (instances)
}
}
}
Expand Down

0 comments on commit 1f24aa0

Please sign in to comment.