[llvm] r204844 - Remove PBQP-cost dimension sanity assertion in PBQP::Graph::addConstructedEdge.

Lang Hames lhames at gmail.com
Wed Mar 26 12:22:51 PDT 2014


Author: lhames
Date: Wed Mar 26 14:22:51 2014
New Revision: 204844

URL: http://llvm.org/viewvc/llvm-project?rev=204844&view=rev
Log:
Remove PBQP-cost dimension sanity assertion in PBQP::Graph::addConstructedEdge.
We're already effectively checking sanity for that in PBQP::Graph::addEdge.


Modified:
    llvm/trunk/include/llvm/CodeGen/PBQP/Graph.h

Modified: llvm/trunk/include/llvm/CodeGen/PBQP/Graph.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/CodeGen/PBQP/Graph.h?rev=204844&r1=204843&r2=204844&view=diff
==============================================================================
--- llvm/trunk/include/llvm/CodeGen/PBQP/Graph.h (original)
+++ llvm/trunk/include/llvm/CodeGen/PBQP/Graph.h Wed Mar 26 14:22:51 2014
@@ -212,13 +212,6 @@ namespace PBQP {
       }
 
       EdgeEntry &NE = getEdge(EId);
-      NodeEntry &N1 = getNode(NE.getN1Id());
-      NodeEntry &N2 = getNode(NE.getN2Id());
-
-      // Sanity check on matrix dimensions:
-      assert((N1.Costs->getLength() == NE.Costs->getRows()) &&
-             (N2.Costs->getLength() == NE.Costs->getCols()) &&
-             "Edge cost dimensions do not match node costs dimensions.");
 
       // Add the edge to the adjacency sets of its nodes.
       NE.connect(*this, EId);





More information about the llvm-commits mailing list