Preserve probe-ness and const-ness in Output() and friends. (#3654)

Using `Output()` shouldn't drop important type information,
only override the specified direction to force alignment.

Until #3647 is addressed, use `cloneTypeFull` to get the expected behavior.
The cloned direction information is unnecessary but is
overridden by the coerced direction.

cc #3647.

Co-authored-by: Jack Koenig <koenig@sifive.com>

Author: dtzSiFive

core/src/main/scala/chisel3/Data.scala

@@ -65,7 +65,7 @@ object SpecifiedDirection {
     val prevId = Builder.idGen.value
     val data = source // evaluate source once (passed by name)
     requireIsChiselType(data)
-    val out = if (!data.mustClone(prevId)) data else data.cloneType.asInstanceOf[T]
+    val out = if (!data.mustClone(prevId)) data else data.cloneTypeFull.asInstanceOf[T]
     out.specifiedDirection = dir(data) // Must use original data, specified direction of clone is cleared
     out
   }

src/test/scala/chiselTests/Direction.scala

@@ -550,4 +550,13 @@ class DirectionSpec extends ChiselPropSpec with Matchers with Utils {
       )
     )
   }
+
+  property("Probe and const are preserved by Output()") {
+    class MyModule extends RawModule {
+      val tpe = probe.Probe(Const(Bool()))
+      val test = IO(Output(tpe))
+    }
+    val emitted: String = ChiselStage.emitCHIRRTL(new MyModule)
+    assert(emitted.contains("Probe<const UInt<1>>"))
+  }
 }