[llvm] r265326 - [ThinLTO] Augment FunctionImport dump with value name to GUID map
Teresa Johnson via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 4 11:52:24 PDT 2016
Author: tejohnson
Date: Mon Apr 4 13:52:23 2016
New Revision: 265326
URL: http://llvm.org/viewvc/llvm-project?rev=265326&view=rev
Log:
[ThinLTO] Augment FunctionImport dump with value name to GUID map
Summary:
To aid in debugging, dump out the correlation between value names and
GUID for each source module when it is materialized. This will make it
easier to comprehend the earlier summary-based function importing debug
trace which only has access to and prints the GUIDs.
Reviewers: joker.eph
Subscribers: llvm-commits, joker.eph
Differential Revision: http://reviews.llvm.org/D18556
Modified:
llvm/trunk/lib/Transforms/IPO/FunctionImport.cpp
Modified: llvm/trunk/lib/Transforms/IPO/FunctionImport.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/IPO/FunctionImport.cpp?rev=265326&r1=265325&r2=265326&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/IPO/FunctionImport.cpp (original)
+++ llvm/trunk/lib/Transforms/IPO/FunctionImport.cpp Mon Apr 4 13:52:23 2016
@@ -320,7 +320,14 @@ bool FunctionImporter::importFunctions(
// Find the globals to import
DenseSet<const GlobalValue *> GlobalsToImport;
for (auto &GV : *SrcModule) {
- if (GV.hasName() && ImportGUIDs.count(GV.getGUID())) {
+ if (!GV.hasName())
+ continue;
+ auto GUID = GV.getGUID();
+ auto Import = ImportGUIDs.count(GUID);
+ DEBUG(dbgs() << (Import ? "Is" : "Not") << " importing " << GUID << " "
+ << GV.getName() << " from " << SrcModule->getSourceFileName()
+ << "\n");
+ if (Import) {
GV.materialize();
GlobalsToImport.insert(&GV);
}
@@ -329,7 +336,11 @@ bool FunctionImporter::importFunctions(
if (!GV.hasName())
continue;
auto GUID = GV.getGUID();
- if (ImportGUIDs.count(GUID)) {
+ auto Import = ImportGUIDs.count(GUID);
+ DEBUG(dbgs() << (Import ? "Is" : "Not") << " importing " << GUID << " "
+ << GV.getName() << " from " << SrcModule->getSourceFileName()
+ << "\n");
+ if (Import) {
// Alias can't point to "available_externally". However when we import
// linkOnceODR the linkage does not change. So we import the alias
// and aliasee only in this case.
@@ -345,7 +356,11 @@ bool FunctionImporter::importFunctions(
if (!GV.hasName())
continue;
auto GUID = GV.getGUID();
- if (ImportGUIDs.count(GUID)) {
+ auto Import = ImportGUIDs.count(GUID);
+ DEBUG(dbgs() << (Import ? "Is" : "Not") << " importing " << GUID << " "
+ << GV.getName() << " from " << SrcModule->getSourceFileName()
+ << "\n");
+ if (Import) {
GV.materialize();
GlobalsToImport.insert(&GV);
}
More information about the llvm-commits
mailing list