[PATCH] D128772: [Lex] Make sure to notify `MultipleIncludeOpt` for "read tokens" during fast dependency directive lexing

Ben Langmuir via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 29 15:05:26 PDT 2022


benlangmuir added inline comments.


================
Comment at: clang/lib/Lex/Lexer.cpp:4251
       DepDirectives.front().Tokens[NextDepDirectiveTokenIndex++];
+  if (NextDepDirectiveTokenIndex > 1 || DDTok.Kind != tok::hash) {
+    // Read something other than a preprocessor directive hash.
----------------
Why do we need the >1 check? I'm not familiar with the details of MIO here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128772



More information about the cfe-commits mailing list