[PATCH] D104344: [modules] Track how headers are included by different modules.

[Volodymyr Sapsai via Phabricator via cfe-commits]

vsapsai added inline comments.


================
Comment at: clang/lib/Serialization/ASTWriter.cpp:1671
       unsigned KeyLen = key.Filename.size() + 1 + 8 + 8;
-      unsigned DataLen = 1 + 2 + 4 + 4;
+      unsigned DataLen = 1 + 4 + 4 + 4;
+      for (auto ModuleIncluder : Data.HFI.ModuleIncludersMap)
----------------
Note for future readers (including myself). Can save 2 bytes because not 
writing `Data.HFI.NumIncludes` but need 4 extra bytes for 
`Data.HFI.ModuleIncludersMap` stop value.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104344

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