[PATCH] D114095: [clang][lex] Include tracking: simplify and move to preprocessor

Volodymyr Sapsai via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 18 16:01:19 PST 2021


vsapsai added inline comments.


================
Comment at: clang/include/clang/Serialization/ASTBitCodes.h:700
+  /// Record code for included files.
+  PP_INCLUDED_FILES = 66,
 };
----------------
Small detail. Do you need to add the new record to `ASTWriter::WriteBlockInfoBlock`? It's not critical but you are one of those who might be grateful later seeing `PP_INCLUDED_FILES` instead of `UnknownCode66` in .pcm dump.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114095



More information about the cfe-commits mailing list