akyrtzi added inline comments.
================ Comment at: clang/lib/Lex/Lexer.cpp:4420 + DepDirectives.front().Tokens[NextDepDirectiveTokenIndex]; + if (BufferStart + NextTok.Offset >= BufferPtr) + break; ---------------- benlangmuir wrote: > How do we know this will terminate? If `LexAngledStringLiteral()` returns `tok::header_name` there were valid tokens that formed a header include and it has set `BufferPtr` after the `>` token. I can't think of reason it would not terminate. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D142143/new/ https://reviews.llvm.org/D142143 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
