From 5e23294dc6ac3c1937c9f071f970178c9f724037 Mon Sep 17 00:00:00 2001
From: Jack Koenig
Date: Fri, 29 Sep 2017 15:43:15 -0700
Subject: StringLit.verilogEscape should support all printable ASCII chars
 (#668)

Defined as the range from ' ' to '~' [0x20, 0x7e]---
 src/main/scala/firrtl/ir/IR.scala           | 8 ++++----
 src/test/scala/firrtlTests/StringSpec.scala | 3 +++
 2 files changed, 7 insertions(+), 4 deletions(-)

(limited to 'src')

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 */
diff --git a/src/test/scala/firrtlTests/StringSpec.scala b/src/test/scala/firrtlTests/StringSpec.scala
index 87ee9191..f1f0bbde 100644
--- a/src/test/scala/firrtlTests/StringSpec.scala
+++ b/src/test/scala/firrtlTests/StringSpec.scala
@@ -66,6 +66,9 @@ class StringSpec extends FirrtlPropSpec {
     val lit = StringLit.unescape(whitelist)
     // Check result
     assert(lit.serialize == whitelist)
+    // Scala likes to escape ' as \', Verilog doesn't
+    val verilogWhitelist = whitelist.replaceAll("""\\'""", "'")
+    assert(lit.verilogEscape.tail.init == verilogWhitelist)
   }
 
   // Valid escapes = \n, \t, \\, \", \'
-- 
cgit v1.2.3