dexonsmith added a comment.

I'm not sure this commit stands on its own, since it introduces a (temporary) 
regression in `.pcm` size; I suggest squashing with the follow-up, 
https://reviews.llvm.org/D114096.



================
Comment at: clang/include/clang/Lex/HeaderSearch.h:55
+  // TODO: Whether the file was imported is not a property of the file itself.
+  // It's a preprocessor state, move it there. 
   /// True if this is a \#import'd file.
----------------
Note that `clang-format` is complaining about a trailing space / ` ` here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114095

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D114095: [... Jan Svoboda via Phabricator via cfe-commits
    • [PATCH] D1140... Duncan P. N. Exon Smith via Phabricator via cfe-commits
    • [PATCH] D1140... Jan Svoboda via Phabricator via cfe-commits

Reply via email to