This has been deprecated for a long time now (and really shouldn't
have existed to begin with).

Deprecating toBits removes the capability to cast an arbitrary
type to UInt.  While it's still possible to do so using asBits.asUInt,
this creates boilerplate.  (asBits is almost never useful itself.)

Remove .Lit(x) usage.
Undo "private" scope change.
Change "firing" back to "fire".
Add package level NODIR definition.

Still fails one test - DirectionSpec in Direction.scala

For Chisel nodes defined in Module class-level values of type Option or
Iterable, we can still use reflection to assign names based on the name
of the value. This works for arbitrary nesting of Option and Iterable so
long as the innermost type is HasId. Note that this excludes Maps which
always have an innermost type of Tuple2[_,_].

Need to convert UInt(x) into UInt.Lit(x) or UInt.width(x)

Compiles but fails tests.

Clean up Scala code, and use +& to generate a lot less FIRRTL

For reasonable circuit delay, need to divide & conquer.