Fix DataMirror.internal.chiselTypeClone to preserve Scala type (#3553) (

#3555) (cherry picked from commit e6c07b3) Co-authored-by: Jack Koenig <[email protected]>
chipsalliance · Sep 26, 2023 · d7630dc · d7630dc
1 parent a4cbe9f
commit d7630dc
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/core/src/main/scala/chisel3/reflect/DataMirror.scala b/core/src/main/scala/chisel3/reflect/DataMirror.scala
@@ -174,8 +174,8 @@ object DataMirror {
   object internal {
     def isSynthesizable(target: Data): Boolean = target.isSynthesizable
     // For those odd cases where you need to care about object reference and uniqueness
-    def chiselTypeClone[T <: Data](target: Data): T = {
-      target.cloneTypeFull.asInstanceOf[T]
+    def chiselTypeClone[T <: Data](target: T): T = {
+      target.cloneTypeFull
     }
   }
 

diff --git a/src/test/scala/chiselTests/reflect/DataMirrorSpec.scala b/src/test/scala/chiselTests/reflect/DataMirrorSpec.scala
@@ -111,4 +111,15 @@ class DataMirrorSpec extends ChiselFlatSpec {
   it should "not support name guesses for non-hardware" in {
     an[ExpectedHardwareException] should be thrownBy DataMirror.queryNameGuess(UInt(8.W))
   }
+
+  "chiselTypeClone" should "preserve Scala type information" in {
+    class MyModule extends Module {
+      val in = IO(Input(UInt(8.W)))
+      val out = IO(Output(DataMirror.internal.chiselTypeClone(in)))
+      // The connection checks the types
+      out :#= in
+    }
+    ChiselStage.emitCHIRRTL(new MyModule)
+  }
+
 }