diff options
| author | Volodymyr Sapsai <vsapsai@apple.com> | 2017-11-17 18:28:05 +0000 |
|---|---|---|
| committer | Volodymyr Sapsai <vsapsai@apple.com> | 2017-11-17 18:28:05 +0000 |
| commit | 8b46ff1648ce2727263e151496ea300d25abdb88 (patch) | |
| tree | c15be921020be43fc3c491cbdf9b22d69c01a6ca /llvm/lib/Analysis/ModuleSummaryAnalysis.cpp | |
| parent | c94d393ad52b6698b15400ee7a33a68b4bda274b (diff) | |
| download | bcm5719-llvm-8b46ff1648ce2727263e151496ea300d25abdb88.tar.gz bcm5719-llvm-8b46ff1648ce2727263e151496ea300d25abdb88.zip | |
[ThinLTO] Remove too aggressive assertion in building function call graph.
The assertion was introduced in r317853 but there are cases when a call
isn't handled either as direct or indirect. In this case we add a
reference graph edge but not a call graph edge.
Reviewers: tejohnson
Reviewed By: tejohnson
Subscribers: mehdi_amini, inglorion, eraman, hiraditya, efriedma, llvm-commits
Differential Revision: https://reviews.llvm.org/D40056
llvm-svn: 318540
Diffstat (limited to 'llvm/lib/Analysis/ModuleSummaryAnalysis.cpp')
| -rw-r--r-- | llvm/lib/Analysis/ModuleSummaryAnalysis.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/llvm/lib/Analysis/ModuleSummaryAnalysis.cpp b/llvm/lib/Analysis/ModuleSummaryAnalysis.cpp index 8d1e7ee770f..d54fb700200 100644 --- a/llvm/lib/Analysis/ModuleSummaryAnalysis.cpp +++ b/llvm/lib/Analysis/ModuleSummaryAnalysis.cpp @@ -280,8 +280,9 @@ computeFunctionSummary(ModuleSummaryIndex &Index, const Module &M, // Skip inline assembly calls. if (CI && CI->isInlineAsm()) continue; - assert(CalledValue && !isa<Constant>(CalledValue) && - "Expected indirect call"); + // Skip direct calls. + if (!CalledValue || isa<Constant>(CalledValue)) + continue; uint32_t NumVals, NumCandidates; uint64_t TotalCount; |
