[llvm-branch-commits] [llvm] [ThinLTO] Generate import status in per-module combined summary (PR #88024)

[Teresa Johnson via llvm-branch-commits]

================
@@ -158,7 +158,7 @@ void llvm::computeLTOCacheKey(
 
   std::vector<uint64_t> ExportsGUID;
   ExportsGUID.reserve(ExportList.size());
-  for (const auto &VI : ExportList) {
+  for (const auto &[VI, UnusedImportType] : ExportList) {
----------------
teresajohnson wrote:

We should probably include the new import type result in the cache key. Because if that changes then presumably the cached object should be invalidated as it would be different?

https://github.com/llvm/llvm-project/pull/88024