[llvm-commits] [llvm] r69152 - /llvm/trunk/utils/TableGen/ClangDiagnosticsEmitter.cpp

[Chris Lattner]

Author: lattner
Date: Wed Apr 15 01:26:49 2009
New Revision: 69152

URL: http://llvm.org/viewvc/llvm-project?rev=69152&view=rev
Log:
minor cleanups

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

Modified: llvm/trunk/utils/TableGen/ClangDiagnosticsEmitter.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/TableGen/ClangDiagnosticsEmitter.cpp?rev=69152&r1=69151&r2=69152&view=diff

==============================================================================
--- llvm/trunk/utils/TableGen/ClangDiagnosticsEmitter.cpp (original)
+++ llvm/trunk/utils/TableGen/ClangDiagnosticsEmitter.cpp Wed Apr 15 01:26:49 2009
@@ -20,7 +20,6 @@
 #include "llvm/ADT/DenseSet.h"
 #include <set>
 #include <map>
-
 using namespace llvm;
 
 //===----------------------------------------------------------------------===//
@@ -68,8 +67,8 @@
 // Warning Tables (.inc file) generation.
 //===----------------------------------------------------------------------===//
 
-static void ProcessDiag(std::ostream& OS, const Record* DiagClass,
-                        const Record& R) {
+static void ProcessDiag(std::ostream &OS, const Record *DiagClass,
+                        const Record &R) {
 
   const Record* DiagKind = getDiagKind(DiagClass, R);
   if (!DiagKind)
@@ -107,17 +106,12 @@
   }
   
   for (RecordVector::const_iterator I=Diags.begin(), E=Diags.end(); I!=E; ++I) {
-    if (!Component.empty()) {
-      const RecordVal* V = findRecordVal(**I, "Component");
-      if (!V)
-        continue;
-
-      const StringInit* SV = dynamic_cast<const StringInit*>(V->getValue());
-      if (!SV || SV->getValue() != Component)
-        continue;
-    }
+    const Record &R = **I;
+    // Filter by component.
+    if (!Component.empty() && Component != R.getValueAsString("Component"))
+      continue;
     
-    ProcessDiag(OS, DiagClass, **I);
+    ProcessDiag(OS, DiagClass, R);
   }
 }
 
@@ -169,8 +163,8 @@
   // If an Option includes another Option, inline the Diagnostics of the
   // included Option.
   if (Def->isSubClassOf("Option")) {
-    if (const RecordVal* V = findRecordVal(*Def, "Members"))
-      if (const ListInit* LV = dynamic_cast<const ListInit*>(V->getValue()))
+    if (const RecordVal *V = findRecordVal(*Def, "Members"))
+      if (const ListInit *LV = dynamic_cast<const ListInit*>(V->getValue()))
         BuildGroup(DS, Visited, LV);
 
     return;
@@ -192,10 +186,10 @@
 
 void ClangOptionsEmitter::run(std::ostream &OS) {
   // Build up a map from options to controlled diagnostics.
-  OptionMap OM;  
+  OptionMap OM;
        
   const RecordVector &Opts = Records.getAllDerivedDefinitions("Option");
-  for (RecordVector::const_iterator I=Opts.begin(), E=Opts.end(); I!=E; ++I)
+  for (RecordVector::const_iterator I=Opts.begin(), E=Opts.end(); I != E; ++I)
     if (const RecordVal* V = findRecordVal(**I, "Members"))
       if (const ListInit* LV = dynamic_cast<const ListInit*>(V->getValue())) {        
         VisitedLists Visited;