Author: rnk Date: Mon Nov 2 14:47:31 2015 New Revision: 251828 URL: http://llvm.org/viewvc/llvm-project?rev=251828&view=rev Log: [PTH] Fix data length used for stat cache entries
This came up in a boost build, which apparently uses PTH. This was broken in r187619 when we migrated it to uses llvm::fs instead of raw stat calls. Constructing a test case with a hash table collision in-tree is tough. Instead, I have a pending change to OnDiskChainedHashTable that asserts that the reported length of the data agrees with the data actually written. All of the existing in-tree tests find the bug with this assert. Modified: cfe/trunk/lib/Frontend/CacheTokens.cpp Modified: cfe/trunk/lib/Frontend/CacheTokens.cpp URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/CacheTokens.cpp?rev=251828&r1=251827&r2=251828&view=diff ============================================================================== --- cfe/trunk/lib/Frontend/CacheTokens.cpp (original) +++ cfe/trunk/lib/Frontend/CacheTokens.cpp Mon Nov 2 14:47:31 2015 @@ -106,7 +106,7 @@ public: } unsigned getRepresentationLength() const { - return Kind == IsNoExist ? 0 : 4 + 4 + 2 + 8 + 8; + return Kind == IsNoExist ? 0 : 4 * 8; } }; _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits