diff options
author | Bob Wilson <bob.wilson@apple.com> | 2012-03-07 23:50:05 +0000 |
---|---|---|
committer | Bob Wilson <bob.wilson@apple.com> | 2012-03-07 23:50:05 +0000 |
commit | f2a23a99db1e5ba454e4f8821bf490ad4bf4d4c1 (patch) | |
tree | f152575e7614be9f547fc685565d7d52f60bc676 /clang/lib/Frontend/CompilerInvocation.cpp | |
parent | 8f89a7b0b83fe3d01aabc52971d7ef2be7225722 (diff) | |
download | bcm5719-llvm-f2a23a99db1e5ba454e4f8821bf490ad4bf4d4c1.tar.gz bcm5719-llvm-f2a23a99db1e5ba454e4f8821bf490ad4bf4d4c1.zip |
Workaround module test failures by removing the version info from module hashes.
PR12196: The module hash strings are not actually hashing the compiler version
string; the entire version string is being included in the hash. Depending on
the module cache directory name, that can lead to failures where the path
names become too long. As a temporary workaround, just remove the version
string from the hash.
llvm-svn: 152266
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
-rw-r--r-- | clang/lib/Frontend/CompilerInvocation.cpp | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/clang/lib/Frontend/CompilerInvocation.cpp b/clang/lib/Frontend/CompilerInvocation.cpp index 7038e85c4a5..9ccd9b23730 100644 --- a/clang/lib/Frontend/CompilerInvocation.cpp +++ b/clang/lib/Frontend/CompilerInvocation.cpp @@ -2196,7 +2196,10 @@ std::string CompilerInvocation::getModuleHash() const { ModuleSignature Signature; // Start the signature with the compiler version. - Signature.add(getClangFullRepositoryVersion()); + // FIXME: The full version string can be quite long. Omit it from the + // module hash for now to avoid failures where the path name becomes too + // long. An MD5 or similar checksum would work well here. + // Signature.add(getClangFullRepositoryVersion()); // Extend the signature with the language options #define LANGOPT(Name, Bits, Default, Description) \ |