[PATCH] D81347: Make ASTFileSignature an array of 20 uint8_t instead of 5 uint32_t

Daniel Grumberg via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Jun 7 12:14:53 PDT 2020


dang marked an inline comment as done.
dang added inline comments.


================
Comment at: clang/include/clang/Basic/Module.h:70
+
+  static ASTFileSignature createDISentinel() {
+    ASTFileSignature Sentinel;
----------------
I couldn't find anywhere in the code base that checks the bit-pattern so I left it as is. If anyone knows if someone could point me to the code that checks this (if any code does check it) that would be nice.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81347





More information about the cfe-commits mailing list