[PATCH] D70056: clang/Modules: Split loop in ReadAST between failable and not

[Duncan P. N. Exon Smith via Phabricator via cfe-commits]

dexonsmith marked an inline comment as done.
dexonsmith added inline comments.


================
Comment at: clang/lib/Serialization/ASTReader.cpp:4236
+                                              MEnd = Loaded.end();
+       M != MEnd; ++M) {
+    ModuleFile &F = *M->Mod;
----------------
aprantl wrote:
> Could this be a range-based for?
Yes, probably.  If so, I'll do that to the existing one as a prep patch (before pushing this change), and follow the style for the new one.


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

https://reviews.llvm.org/D70056