jansvoboda11 added inline comments.
================ Comment at: clang/lib/Frontend/FrontendAction.cpp:444 FileID ModuleMapID = SrcMgr.getMainFileID(); - const FileEntry *ModuleMap = SrcMgr.getFileEntryForID(ModuleMapID); + Optional<FileEntryRef> ModuleMap = SrcMgr.getFileEntryRefForID(ModuleMapID); ---------------- bnbarham wrote: > The empty case isn't checked as far as I can tell. The null case wasn't > either, but seems like we should just `return true`? Good point. I think the entry for `MainFileID` always exists, so I'll add an assert here. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D127647/new/ https://reviews.llvm.org/D127647 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
