Add (hidden) TableGen command option '-clang-component' which specifies the
authorTed Kremenek <kremenek@apple.com>
Fri, 13 Mar 2009 22:53:41 +0000 (22:53 +0000)
committerTed Kremenek <kremenek@apple.com>
Fri, 13 Mar 2009 22:53:41 +0000 (22:53 +0000)
component's warnings to process for '-gen-clang-diags-defs'.

Also, when the component is specified, generate a '#if' prologue at the top of
the generated .def file (to match the current files).

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@66975 91177308-0d34-0410-b5e6-96231b3b80d8

utils/TableGen/ClangDiagnosticsEmitter.cpp
utils/TableGen/ClangDiagnosticsEmitter.h
utils/TableGen/TableGen.cpp

index fa139e7729e6116508bbe263d3c95b8b0ccf400e..f9501fb494dad61500118bdf0aed151dd80d62f9 100644 (file)
@@ -22,8 +22,8 @@ typedef std::vector<Record*> RecordVector;
 typedef std::vector<Record*> SuperClassVector;
 typedef std::vector<RecordVal> RecordValVector;
 
-static const RecordVal* findRecordVal(const RecordValVector& Vals,
-                                      const std::string &key) {
+static const RecordVal* findRecordVal(const Record& R, const std::string &key) {  
+  const RecordValVector &Vals = R.getValues();
   for (RecordValVector::const_iterator I=Vals.begin(), E=Vals.end(); I!=E; ++I)
     if ((*I).getName() == key)
       return &*I;
@@ -49,6 +49,11 @@ static void EmitEscaped(std::ostream& OS, const std::string &s) {
     }
 }
 
+static void EmitAllCaps(std::ostream& OS, const std::string &s) {
+  for (std::string::const_iterator I=s.begin(), E=s.end(); I!=E; ++I)
+    OS << char(toupper(*I));  
+}
+
 static void ProcessDiag(std::ostream& OS, const Record* DiagClass,
                         const Record& R) {
 
@@ -57,12 +62,9 @@ static void ProcessDiag(std::ostream& OS, const Record* DiagClass,
     return;
 
   OS << "DIAG(" << R.getName() << ", ";
+  EmitAllCaps(OS, DiagKind->getName());
   
-  const std::string &s = DiagKind->getName();
-  for (std::string::const_iterator I=s.begin(), E=s.end(); I!=E; ++I)
-    OS << char(toupper(*I));
-  
-  const RecordVal* Text = findRecordVal(R.getValues(), "Text");
+  const RecordVal* Text = findRecordVal(R, "Text");
   assert(Text && "No 'Text' entry in Diagnostic.");
   const StringInit* TextVal = dynamic_cast<const StringInit*>(Text->getValue());
   assert(TextVal && "Value 'Text' must be a string.");
@@ -77,8 +79,30 @@ void ClangDiagsDefsEmitter::run(std::ostream &OS) {
   const Record* DiagClass = Records.getClass("Diagnostic");
   assert(DiagClass && "No Diagnostic class defined.");  
   
+  // Write the #if guard
+  if (!Component.empty()) {
+    OS << "#ifdef ";
+    EmitAllCaps(OS, Component);
+    OS << "START\n__";
+    EmitAllCaps(OS, Component);
+    OS << "START = DIAG_START_";
+    EmitAllCaps(OS, Component);
+    OS << ",\n#undef ";
+    EmitAllCaps(OS, Component);
+    OS << "START\n#endif\n";
+  }
+  
   for (RecordVector::const_iterator I=Diags.begin(), E=Diags.end(); I!=E; ++I) {
-    // FIXME: Compare the component.
+    if (!Component.empty()) {
+      const RecordVal* V = findRecordVal(**I, "Component");
+      if (!V)
+        continue;
+
+      const StringInit* SV = dynamic_cast<const StringInit*>(V->getValue());
+      if (SV->getValue() != Component)
+        continue;
+    }
+    
     ProcessDiag(OS, DiagClass, **I);
   }
-}
\ No newline at end of file
+}
index 93979708b0789f3a464a2f901ba20e724f8d6284..bedfe5af8f8e65cecb7f9a731e449a47abd549ea 100644 (file)
@@ -23,8 +23,10 @@ namespace llvm {
 ///
 class ClangDiagsDefsEmitter : public TableGenBackend {
   RecordKeeper &Records;
+  const std::string& Component;
 public:
-  explicit ClangDiagsDefsEmitter(RecordKeeper &R) : Records(R) {}
+  explicit ClangDiagsDefsEmitter(RecordKeeper &R, const std::string& component)
+    : Records(R), Component(component) {}
 
   // run - Output the .def file contents
   void run(std::ostream &OS);
index e6ed092b62fce35664d400a4c18f0d3e17b87876..22c5d50415544d6f446a1e4136d1fde952c9e341 100644 (file)
@@ -110,6 +110,11 @@ namespace {
   cl::list<std::string>
   IncludeDirs("I", cl::desc("Directory of include files"),
               cl::value_desc("directory"), cl::Prefix);
+  
+  cl::opt<std::string>
+  ClangComponent("clang-component",
+                 cl::desc("Only use warnings from specified component"),
+                 cl::value_desc("component"), cl::Hidden);
 }
 
 
@@ -202,7 +207,7 @@ int main(int argc, char **argv) {
       AsmWriterEmitter(Records).run(*Out);
       break;
     case GenClangDiagsDefs:
-      ClangDiagsDefsEmitter(Records).run(*Out);
+      ClangDiagsDefsEmitter(Records, ClangComponent).run(*Out);
       break;
     case GenDAGISel:
       DAGISelEmitter(Records).run(*Out);