Remove redundancy code (#1296) (#1312)

* Remove redundancy code * Remove blank line * BitPat supports whitespace and underscores, presumably for human readability. The BitPat.parse factory though did not remove these from the returned count. This fixes that adds whitespace and underscores to the unit tests This is an updated vesion of Chisel PR #1069 Co-authored-by: Chick Markley <[email protected]> Co-authored-by: Jim Lawson <[email protected]> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> (cherry picked from commit 160e019) Co-authored-by: Leway Colin <[email protected]>
chipsalliance · Jan 25, 2020 · b710470 · b710470
1 parent 3c20ed9
commit b710470
Showing 1 changed file with 0 additions and 11 deletions.
diff --git a/chiselFrontend/src/main/scala/chisel3/RawModule.scala b/chiselFrontend/src/main/scala/chisel3/RawModule.scala
@@ -184,17 +184,6 @@ package internal {
     // Allow access to bindings from the compatibility package
     protected def _compatIoPortBound() = portsContains(io)// scalastyle:ignore method.name
 
-    protected override def nameIds(rootClass: Class[_]): HashMap[HasId, String] = {
-      val names = super.nameIds(rootClass)
-
-      // Allow IO naming without reflection
-      names.put(io, "io")
-      names.put(clock, "clock")
-      names.put(reset, "reset")
-
-      names
-    }
-
     private[chisel3] override def namePorts(names: HashMap[HasId, String]): Unit = {
       for (port <- getModulePorts) {
         // This should already have been caught