StringLit.verilogEscape should support all printable ASCII chars (#668)

Defined as the range from ' ' to '~' [0x20, 0x7e]

1 files changed, 4 insertions, 4 deletions

diff --git a/src/main/scala/firrtl/ir/IR.scala b/src/main/scala/firrtl/ir/IR.scala
index 2f37b05a..f2c39f8d 100644
--- a/src/main/scala/firrtl/ir/IR.scala
+++ b/src/main/scala/firrtl/ir/IR.scala
@@ -69,12 +69,12 @@ object StringLit {
   /** Maps characters to ASCII for Verilog emission */
   private def toASCII(char: Char): List[Char] = char match {
     case nonASCII if !nonASCII.isValidByte => List('?')
-    case letter if letter.isLetter => List(letter)
-    case '\n' => List('\\', 'n')
+    case '"' => List('\\', '"')
     case '\\' => List('\\', '\\')
+    case c if c >= ' ' && c <= '~' => List(c)
+    case '\n' => List('\\', 'n')
     case '\t' => List('\\', 't')
-    case '"' => List('\\', '"')
-    case other => List('?')
+    case _ => List('?')
   }
 
   /** Create a StringLit from a raw parsed String */