Better handling of insertion points when no diagnostics are present

Author: cmaglie

main.cpp

@@ -160,10 +160,20 @@ class INOPreprocessorMatcherCallback : public MatchFinder::MatchCallback {
     }
 
     void detectInsertionPoint(SourceManager &sm, FullSourceLoc &begin, FullSourceLoc &end) {
-        if (insertionPointFound)return;
+        if (insertionPointFound) {
+            return;
+        }
 
-        FullSourceLoc first = undeclaredIdentifiers.front()->location;
+        if (undeclaredIdentifiers.empty()) {
+            //insertionPoint = begin;
+            //insertionPointFound = true;
+            //if (debugOutput) {
+            //    outs() << "  !! Insertion point found (using the first available position)\n";
+            //}
+            return;
+        }
 
+        FullSourceLoc first = undeclaredIdentifiers.back()->location;
         if (first.isBeforeInTranslationUnitThan(begin)) {
             if (debugOutput) {
                 outs() << "  !! Insertion point found!\n";
@@ -172,6 +182,10 @@ class INOPreprocessorMatcherCallback : public MatchFinder::MatchCallback {
             return;
         }
 
+        if (end.isInvalid()) {
+            return;
+        }
+
         insertionPoint = begin;
         presumedInsertionPoint = sm.getPresumedLoc(begin, true);
         if (debugOutput) {