This revision was automatically updated to reflect the committed changes.
Closed by commit rG64a390c1bc75: Modules: Remove an extra early return, NFC 
(authored by dexonsmith).

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101671

Files:
  clang/lib/Frontend/CompilerInstance.cpp


Index: clang/lib/Frontend/CompilerInstance.cpp
===================================================================
--- clang/lib/Frontend/CompilerInstance.cpp
+++ clang/lib/Frontend/CompilerInstance.cpp
@@ -1890,8 +1890,6 @@
       return Result;
     Module = Result;
     MM.cacheModuleLoad(*Path[0].first, Module);
-    if (!Module)
-      return Module;
   }
 
   // If we never found the module, fail.  Otherwise, verify the module and link


Index: clang/lib/Frontend/CompilerInstance.cpp
===================================================================
--- clang/lib/Frontend/CompilerInstance.cpp
+++ clang/lib/Frontend/CompilerInstance.cpp
@@ -1890,8 +1890,6 @@
       return Result;
     Module = Result;
     MM.cacheModuleLoad(*Path[0].first, Module);
-    if (!Module)
-      return Module;
   }
 
   // If we never found the module, fail.  Otherwise, verify the module and link
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D101671: M... Duncan P. N. Exon Smith via Phabricator via cfe-commits
    • [PATCH] D1016... David Blaikie via Phabricator via cfe-commits
    • [PATCH] D1016... Duncan P. N. Exon Smith via Phabricator via cfe-commits

Reply via email to