Properly Iterate Through RequestedCachedResults

Clearing the cache and the end of the for loop invalidates the interator
and prevents iterating through the rest of the vector. This should be
cleared after we're done iterating.

Author: gmittert

lib/IDE/CodeCompletion.cpp

@@ -5568,8 +5568,8 @@ void CodeCompletionCallbacksImpl::doneParsing() {
         TheModule->forAllVisibleModules(AccessPath, handleImport);
       }
     }
-    Lookup.RequestedCachedResults.clear();
   }
+  Lookup.RequestedCachedResults.clear();
 
   CompletionContext.typeContextKind = Lookup.typeContextKind();