[llvm-commits] CVS: llvm/utils/TableGen/DAGISelEmitter.cpp

[Evan Cheng]

Changes in directory llvm/utils/TableGen:

DAGISelEmitter.cpp updated: 1.233 -> 1.234
---
Log message:

Pass N by reference to select functions to prevent gcc from allocating more stack.

---
Diffs of the changes:  (+3 -3)

 DAGISelEmitter.cpp |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)


Index: llvm/utils/TableGen/DAGISelEmitter.cpp
diff -u llvm/utils/TableGen/DAGISelEmitter.cpp:1.233 llvm/utils/TableGen/DAGISelEmitter.cpp:1.234
--- llvm/utils/TableGen/DAGISelEmitter.cpp:1.233	Mon Jul 31 14:01:58 2006
+++ llvm/utils/TableGen/DAGISelEmitter.cpp	Mon Jul 31 20:07:48 2006
@@ -1829,7 +1829,7 @@
   // If this node is commutative, consider the commuted order.
   if (NodeInfo.hasProperty(SDNodeInfo::SDNPCommutative)) {
     assert(N->getNumChildren()==2 &&"Commutative but doesn't have 2 children!");
-    // Don't count childrean which are actually
+    // Don't count children which are actually register references.
     unsigned NC = 0;
     for (unsigned i = 0, e = N->getNumChildren(); i != e; ++i) {
       TreePatternNode *Child = N->getChild(i);
@@ -3321,7 +3321,7 @@
         }
 
         std::string CalleeDecls;
-        std::string CalleeCode = "(SDOperand &Result, SDOperand &N";
+        std::string CalleeCode = "(SDOperand &Result, const SDOperand &N";
         std::string CallerCode = "(Result, N";
         for (unsigned j = 0, e = TargetOpcodes.size(); j != e; ++j) {
           CalleeCode += ", unsigned Opc" + utostr(j);
@@ -3399,7 +3399,7 @@
         OpVTI->second.push_back(OpVTStr);
 
       OS << "void Select_" << OpName << (OpVTStr != "" ? "_" : "")
-         << OpVTStr << "(SDOperand &Result, SDOperand N) {\n";    
+         << OpVTStr << "(SDOperand &Result, const SDOperand &N) {\n";    
       if (OptSlctOrder) {
         OS << "  if (N.ResNo == " << OpcodeInfo.getNumResults()
            << " && N.getValue(0).hasOneUse()) {\n"