diff options
author | Teresa Johnson <tejohnson@google.com> | 2016-04-11 13:58:45 +0000 |
---|---|---|
committer | Teresa Johnson <tejohnson@google.com> | 2016-04-11 13:58:45 +0000 |
commit | 2d5487cf446f9a58f6d2f76df33102423c83f285 (patch) | |
tree | 43c13994a51a13597dcc383cf013614f4c88c2b7 /llvm/lib/Transforms/IPO/FunctionImport.cpp | |
parent | 34ffcbbe8446eb4092bb83760fc38ad756bd0a6a (diff) | |
download | bcm5719-llvm-2d5487cf446f9a58f6d2f76df33102423c83f285.tar.gz bcm5719-llvm-2d5487cf446f9a58f6d2f76df33102423c83f285.zip |
[ThinLTO] Move summary computation from BitcodeWriter to new pass
Summary:
This is the first step in also serializing the index out to LLVM
assembly.
The per-module summary written to bitcode is moved out of the bitcode
writer and to a new analysis pass (ModuleSummaryIndexWrapperPass).
The pass itself uses a new builder class to compute index, and the
builder class is used directly in places where we don't have a pass
manager (e.g. llvm-as).
Because we are computing summaries outside of the bitcode writer, we no
longer can use value ids created by the bitcode writer's
ValueEnumerator. This required changing the reference graph edge type
to use a new ValueInfo class holding a union between a GUID (combined
index) and Value* (permodule index). The Value* are converted to the
appropriate value ID during bitcode writing.
Also, this enables removal of the BitWriter library's dependence on the
Analysis library that was previously required for the summary computation.
Reviewers: joker.eph
Subscribers: joker.eph, llvm-commits
Differential Revision: http://reviews.llvm.org/D18763
llvm-svn: 265941
Diffstat (limited to 'llvm/lib/Transforms/IPO/FunctionImport.cpp')
-rw-r--r-- | llvm/lib/Transforms/IPO/FunctionImport.cpp | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/llvm/lib/Transforms/IPO/FunctionImport.cpp b/llvm/lib/Transforms/IPO/FunctionImport.cpp index 32853b93f28..63b2e974419 100644 --- a/llvm/lib/Transforms/IPO/FunctionImport.cpp +++ b/llvm/lib/Transforms/IPO/FunctionImport.cpp @@ -145,7 +145,7 @@ static void computeImportForFunction( FunctionImporter::ImportMapTy &ImportsForModule, StringMap<FunctionImporter::ExportSetTy> &ExportLists) { for (auto &Edge : Summary.calls()) { - auto GUID = Edge.first; + auto GUID = Edge.first.getGUID(); DEBUG(dbgs() << " edge -> " << GUID << " Threshold:" << Threshold << "\n"); if (DefinedFunctions.count(GUID)) { @@ -181,11 +181,12 @@ static void computeImportForFunction( // Mark all functions and globals referenced by this function as exported to // the outside if they are defined in the same source module. for (auto &Edge : CalleeSummary->calls()) { - auto CalleeGUID = Edge.first; + auto CalleeGUID = Edge.first.getGUID(); if (isGlobalExported(Index, ExportModulePath, CalleeGUID)) ExportList.insert(CalleeGUID); } - for (auto &GUID : CalleeSummary->refs()) { + for (auto &Ref : CalleeSummary->refs()) { + auto GUID = Ref.getGUID(); if (isGlobalExported(Index, ExportModulePath, GUID)) ExportList.insert(GUID); } |