https://github.com/Bigcheese closed
https://github.com/llvm/llvm-project/pull/73719
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ributzka approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/73719
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/benlangmuir approved this pull request.
https://github.com/llvm/llvm-project/pull/73719
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -325,6 +326,13 @@ static std::string getModuleContextHash(const ModuleDeps
,
// will be readable.
HashBuilder.add(getClangFullRepositoryVersion());
HashBuilder.add(serialization::VERSION_MAJOR, serialization::VERSION_MINOR);
+ if
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Michael Spencer (Bigcheese)
Changes
The working directory is included in the PCM, but is not currently part of the
context hash. This causes problems because different builds of a PCM with
exactly the same command line can end up with
https://github.com/Bigcheese created
https://github.com/llvm/llvm-project/pull/73719
The working directory is included in the PCM, but is not currently part of the
context hash. This causes problems because different builds of a PCM with
exactly the same command line can end up with different