[PATCH] D64753: [CrossTU][NFCI] Refactor loadExternalAST function

Gabor Marton via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 31 05:41:19 PDT 2019


martong added inline comments.


================
Comment at: clang/lib/CrossTU/CrossTranslationUnit.cpp:489
+  if (DisplayCTUProgress) {
+    if (llvm::Expected<std::string> FileName =
+            ASTStorage.getFileForFunction(LookupName, CrossTUDir, IndexName))
----------------
` LoadOperation` should not be successful if the condition here is true.
Perhaps `LoadOperation.wasSuccessful();` should be done right before the last return stmt in the function.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64753/new/

https://reviews.llvm.org/D64753





More information about the llvm-commits mailing list