Revision: 23622
Author:   [email protected]
Date:     Tue Sep  2 14:59:41 2014 UTC
Log:      Fix bug in graph visualizer.

[email protected]

Review URL: https://codereview.chromium.org/526373002
https://code.google.com/p/v8/source/detail?r=23622

Modified:
 /branches/bleeding_edge/src/compiler/graph-visualizer.cc

=======================================
--- /branches/bleeding_edge/src/compiler/graph-visualizer.cc Tue Sep 2 08:44:25 2014 UTC +++ /branches/bleeding_edge/src/compiler/graph-visualizer.cc Tue Sep 2 14:59:41 2014 UTC
@@ -209,14 +209,14 @@
              (OperatorProperties::GetControlInputCount(from->op()) > 0 &&
               NodeProperties::GetControlInput(from) != to)) {
     os_ << ":I" << index << ":n -> ID" << to->id() << ":s"
-        << "[" << (unconstrained ? "constraint=false" : "")
-        << (NodeProperties::IsControlEdge(edge) ? "style=bold" : "")
-        << (NodeProperties::IsEffectEdge(edge) ? "style=dotted" : "")
- << (NodeProperties::IsContextEdge(edge) ? "style=dashed" : "") << "]";
+        << "[" << (unconstrained ? "constraint=false, " : "")
+        << (NodeProperties::IsControlEdge(edge) ? "style=bold, " : "")
+        << (NodeProperties::IsEffectEdge(edge) ? "style=dotted, " : "")
+ << (NodeProperties::IsContextEdge(edge) ? "style=dashed, " : "") << "]";
   } else {
-    os_ << " -> ID" << to->id() << ":s [color=transparent"
-        << (unconstrained ? ", constraint=false" : "")
- << (NodeProperties::IsControlEdge(edge) ? ", style=dashed" : "") << "]";
+    os_ << " -> ID" << to->id() << ":s [color=transparent, "
+        << (unconstrained ? "constraint=false, " : "")
+ << (NodeProperties::IsControlEdge(edge) ? "style=dashed, " : "") << "]";
   }
   os_ << "\n";
 }

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to