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

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

Reply via email to