[PATCH] D66511: [clang-scan-deps] Skip UTF-8 BOM in source minimizer

Duncan P. N. Exon Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 23 10:10:52 PDT 2019


dexonsmith added inline comments.


================
Comment at: lib/Lex/DependencyDirectivesSourceMinimizer.cpp:822
 bool Minimizer::minimizeImpl(const char *First, const char *const End) {
+  skipUTF8ByteOrderMark(First, End);
   while (First != End)
----------------
Is skipping this the right thing, or should it also be copied to the output?


Repository:
  rC Clang

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

https://reviews.llvm.org/D66511





More information about the cfe-commits mailing list