[PATCH] D134249: [modules] Fix error "malformed or corrupted AST file: 'SourceLocation remap refers to unknown module...'".

Volodymyr Sapsai via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 20 17:57:23 PDT 2022


vsapsai marked an inline comment as done.
vsapsai added a comment.

Thanks for the review!



================
Comment at: clang/lib/Serialization/ModuleManager.cpp:284-286
   for (ModuleIterator victim = First; victim != Last; ++victim) {
     Modules.erase(victim->File);
   }
----------------
ChuanqiXu wrote:
> minor: we can remove the `{` here.
Done in actual commit.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134249



More information about the cfe-commits mailing list