[PATCH] D65906: [clang-scan-deps] Fix edge cases in the minimizer

Alex Lorenz via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 8 18:45:51 PDT 2019


arphaman added a comment.

@aganea These are not just any invisible characters that you have, this is the UTF8 BOM. Clang's Lexer skips over them if they're in the beginning of the file (`Lexer::InitLexer`). The minimizer should do the same thing, so ideally you would factor out the BOM detection and teach the minimizer to skip past it.


Repository:
  rC Clang

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

https://reviews.llvm.org/D65906





More information about the cfe-commits mailing list