[llvm-commits] CVS: llvm/lib/Analysis/DataStructure/DataStructure.cpp

Chris Lattner lattner at cs.uiuc.edu
Thu Jul 8 02:27:00 PDT 2004


Changes in directory llvm/lib/Analysis/DataStructure:

DataStructure.cpp updated: 1.177 -> 1.178

---
Log message:

Disable some code that isn't helping matters


---
Diffs of the changes:  (+6 -1)

Index: llvm/lib/Analysis/DataStructure/DataStructure.cpp
diff -u llvm/lib/Analysis/DataStructure/DataStructure.cpp:1.177 llvm/lib/Analysis/DataStructure/DataStructure.cpp:1.178
--- llvm/lib/Analysis/DataStructure/DataStructure.cpp:1.177	Wed Jul  7 01:32:21 2004
+++ llvm/lib/Analysis/DataStructure/DataStructure.cpp	Thu Jul  8 02:25:51 2004
@@ -1589,20 +1589,25 @@
 void DSGraph::removeTriviallyDeadNodes() {
   TIME_REGION(X, "removeTriviallyDeadNodes");
 
+#if 0
+  /// NOTE: This code is disabled.  This slows down DSA on 177.mesa
+  /// substantially!
+
   // Loop over all of the nodes in the graph, calling getNode on each field.
   // This will cause all nodes to update their forwarding edges, causing
   // forwarded nodes to be delete-able.
+  { TIME_REGION(X, "removeTriviallyDeadNodes:node_iterate");
   for (node_iterator NI = node_begin(), E = node_end(); NI != E; ++NI) {
     DSNode *N = *NI;
     for (unsigned l = 0, e = N->getNumLinks(); l != e; ++l)
       N->getLink(l*N->getPointerSize()).getNode();
   }
+  }
 
   // NOTE: This code is disabled.  Though it should, in theory, allow us to
   // remove more nodes down below, the scan of the scalar map is incredibly
   // expensive for certain programs (with large SCCs).  In the future, if we can
   // make the scalar map scan more efficient, then we can reenable this.
-#if 0
   { TIME_REGION(X, "removeTriviallyDeadNodes:scalarmap");
 
   // Likewise, forward any edges from the scalar nodes.  While we are at it,





More information about the llvm-commits mailing list