dexonsmith added inline comments.
================ Comment at: clang/lib/Lex/HeaderSearch.cpp:305 // maps, searching for a module map that describes this module. - for (It = search_dir_begin(); It != search_dir_end(); ++It) { - if (It->isFramework()) { + for (DirectoryLookup Dir : search_dir_range()) { + if (Dir.isFramework()) { ---------------- This triggers a bunch of noise in this patch. Is it possible to make this change either before or after, to avoid the noise, giving that there's a functional change hiding here? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121295/new/ https://reviews.llvm.org/D121295 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits