Do not pollute output with 'non-toplevel-identifiers' in debug mode.

cmaglie · cmaglie · commit f195c49cc9cf · 2017-08-07T15:30:54.000+02:00
diff --git a/main.cpp b/main.cpp
@@ -142,6 +142,13 @@ class INOPreprocessorMatcherCallback : public MatchFinder::MatchCallback {
 
         const VarDecl *v = match.Nodes.getNodeAs<VarDecl>("var_decl");
         if (v) {
+            if (v->getParentFunctionOrMethod()) {
+                //if (debugOutput) {
+                //    outs() << "  Variable is not top level, ignoring.\n";
+                //}
+                return;
+            }
+
             FullSourceLoc loc = ctx->getFullLoc(v->getLocStart());
             SourceRange r = v->getSourceRange();
             FullSourceLoc begin = ctx->getFullLoc(r.getBegin());
@@ -153,12 +160,6 @@ class INOPreprocessorMatcherCallback : public MatchFinder::MatchCallback {
                 outs() << " (range " << begin.getSpellingLineNumber() << ":" << begin.getSpellingColumnNumber();
                 outs() << " to " << end.getSpellingLineNumber() << ":" << end.getSpellingColumnNumber() << ")\n";
             }
-            if (v->getParentFunctionOrMethod()) {
-                if (debugOutput) {
-                    outs() << "  Variable is not top level, ignoring.\n";
-                }
-                return;
-            }
 
             detectInsertionPoint(sm, begin, end);
         }
