sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land.
LG, though I'm obviously not an expert here. Thanks for tracking this down! ================ Comment at: clang/lib/Lex/PPDirectives.cpp:2405 + auto _ = llvm::make_scope_exit([&]() { + // The flag indicates if we have already reached 'eod'. + if (CurLexer->ParsingPreprocessorDirective) ---------------- nit: "if we are still waiting for eod"? (to avoid inverting the sense) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65517/new/ https://reviews.llvm.org/D65517 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits