Restore "[ThinLTO] Ensure we always select the same function copy to import"
authorTeresa Johnson <tejohnson@google.com>
Mon, 16 Jul 2018 15:30:36 +0000 (15:30 +0000)
committerTeresa Johnson <tejohnson@google.com>
Mon, 16 Jul 2018 15:30:36 +0000 (15:30 +0000)
This reverts commit r337082, restoring r337051, since the LLVM side
patch has been restored.

llvm-svn: 337185

clang/lib/CodeGen/BackendUtil.cpp

index 2a02b83..226a27b 100644 (file)
@@ -1127,9 +1127,8 @@ static void runThinLTOBackend(ModuleSummaryIndex *CombinedIndex, Module *M,
     // e.g. record required linkage changes.
     if (Summary->modulePath() == M->getModuleIdentifier())
       continue;
-    // Doesn't matter what value we plug in to the map, just needs an entry
-    // to provoke importing by thinBackend.
-    ImportList[Summary->modulePath()][GUID] = 1;
+    // Add an entry to provoke importing by thinBackend.
+    ImportList[Summary->modulePath()].insert(GUID);
   }
 
   std::vector<std::unique_ptr<llvm::MemoryBuffer>> OwnedImports;