Serializer: only serialize smem RUW if non-default (#2597)

This will make it easier to compile circuits with older compiler versions.
chipsalliance · Jan 25, 2023 · 84a7db5 · 84a7db5
1 parent 82af22f
commit 84a7db5
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/main/scala/firrtl/ir/Serializer.scala b/src/main/scala/firrtl/ir/Serializer.scala
@@ -303,7 +303,10 @@ object Serializer {
     case firrtl.CDefMemory(info, name, tpe, size, seq, readUnderWrite) =>
       if (seq) b ++= "smem " else b ++= "cmem "
       b ++= name; b ++= " : "; s(tpe); b ++= " ["; b ++= size.toString(); b += ']'
-      b += ' '; b ++= readUnderWrite.toString; s(info)
+      if (readUnderWrite != ReadUnderWrite.Undefined) { // undefined is the default
+        b += ' '; b ++= readUnderWrite.toString
+      }
+      s(info)
     case firrtl.CDefMPort(info, name, _, mem, exps, direction) =>
       b ++= direction.serialize; b ++= " mport "; b ++= name; b ++= " = "; b ++= mem
       b += '['; s(exps.head); b ++= "], "; s(exps(1)); s(info)