[llvm-commits] [llvm] r96422 - /llvm/trunk/utils/TableGen/DAGISelMatcherEmitter.cpp

[Chris Lattner]

Author: lattner
Date: Tue Feb 16 18:39:26 2010
New Revision: 96422

URL: http://llvm.org/viewvc/llvm-project?rev=96422&view=rev
Log:
improve comments in generated matcher a bit.

Modified:
    llvm/trunk/utils/TableGen/DAGISelMatcherEmitter.cpp

Modified: llvm/trunk/utils/TableGen/DAGISelMatcherEmitter.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/TableGen/DAGISelMatcherEmitter.cpp?rev=96422&r1=96421&r2=96422&view=diff

==============================================================================
--- llvm/trunk/utils/TableGen/DAGISelMatcherEmitter.cpp (original)
+++ llvm/trunk/utils/TableGen/DAGISelMatcherEmitter.cpp Tue Feb 16 18:39:26 2010
@@ -21,7 +21,7 @@
 
 namespace {
 enum {
-  CommentIndent = 25
+  CommentIndent = 30
 };
 }
 
@@ -119,11 +119,11 @@
   switch (N->getKind()) {
   case MatcherNode::Push: assert(0 && "Should be handled by caller");
   case MatcherNode::EmitNode:
-    OS << "OPC_Emit, /*XXX*/";
-    OS.PadToColumn(CommentIndent) << "// Src: "
-      << *cast<EmitNodeMatcherNode>(N)->getPattern().getSrcPattern() << '\n';
-    OS.PadToColumn(CommentIndent) << "// Dst: "
-      << *cast<EmitNodeMatcherNode>(N)->getPattern().getDstPattern() << '\n';
+    OS << "// Src: "
+       << *cast<EmitNodeMatcherNode>(N)->getPattern().getSrcPattern() << '\n';
+    OS.PadToColumn(Indent*2) << "// Dst: "
+       << *cast<EmitNodeMatcherNode>(N)->getPattern().getDstPattern() << "\n";
+    OS.PadToColumn(Indent*2) << "OPC_Emit, /*XXX*/\n\n";
     return 1;
   case MatcherNode::Record:
     OS << "OPC_Record,\n";
@@ -180,11 +180,13 @@
        << cast<CheckValueTypeMatcherNode>(N)->getTypeName() << ",\n";
     return 2;
 
-  case MatcherNode::CheckComplexPat:
-    OS << "OPC_CheckComplexPat, "
-       << getComplexPat(cast<CheckComplexPatMatcherNode>(N)->getPattern())
-       << ",\n";
+  case MatcherNode::CheckComplexPat: {
+    const ComplexPattern &Pattern =
+      cast<CheckComplexPatMatcherNode>(N)->getPattern();
+    OS << "OPC_CheckComplexPat, " << getComplexPat(Pattern) << ',';
+    OS.PadToColumn(CommentIndent) << "// " << Pattern.getSelectFunc() << '\n';
     return 2;
+  }
       
   case MatcherNode::CheckAndImm: {
     int64_t Val = cast<CheckAndImmMatcherNode>(N)->getValue();
@@ -271,7 +273,6 @@
   OS << "}\n\n";
   
   // Emit CompletePattern matchers.
-  
   OS << "bool CheckComplexPattern(SDNode *Root, SDValue N,\n";
   OS << "      unsigned PatternNo, SmallVectorImpl<SDValue> &Result) {\n";
   OS << "  switch (PatternNo) {\n";
@@ -290,7 +291,6 @@
   }
   OS << "  }\n";
   OS << "}\n\n";
-  
 }