akyrtzi 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.
----------------
benlangmuir wrote:
> Why do we need the >1 check? I'm not familiar with the details of MIO here.
I'm making sure that `MIOpt.ReadToken()` is called for every token except the 
starting hash of a preprocessor directive, to match what regular lexing is 
doing.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128772

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to