[PATCH] D67249: [Modules][PCH] Hash input files content
Adrian Prantl via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Sep 5 18:55:33 PDT 2019
aprantl added a comment.
Nice! Are you planning to address the FIXME's in a later update of this patch?
================
Comment at: clang/lib/Serialization/ASTWriter.cpp:1767
bool IsTopLevelModuleMap;
+ uint32_t ContentHash[2];
};
----------------
Why is this not a uint64_t?
================
Comment at: clang/test/Modules/validate-file-content.m:14
+// RUN: %clang_cc1 -emit-pch -fmodules-cache-path=%t/cache -fmodules -fimplicit-module-maps -o %t/a.pch -I %t -x objective-c-header %t/a.h -fvalidate-ast-input-files-content
+// RUN: touch -m -a -A 01 %t/m.h
+// RUN: %clang_cc1 -fsyntax-only -fmodules-cache-path=%t/cache -fmodules -fimplicit-module-maps -I %t -include-pch %t/a.pch %s -verify -fvalidate-ast-input-files-content
----------------
Are there other tests that use touch and file timestamps in general? I wonder if this could cause random issues when building on NFS or filesystems with a really low timestamp granularity. I don't have a better suggestion either since the mtime is part of the code being tested here.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D67249/new/
https://reviews.llvm.org/D67249
More information about the cfe-commits
mailing list