RPCS3/llvm-mirror
1
0
Fork 0
mirror of https://github.com/RPCS3/llvm-mirror.git synced 2024-12-04 09:45:00 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

[ModuleSummaryAnalysis] Don't crash when referencing unnamed globals.

Browse Source 
Instead, just be conservative as these are unfrequent enough. Thanks
to Peter Collingbourne for the discussion about this on IRC.

llvm-svn: 295861
This commit is contained in:
Davide Italiano 2017-02-22 18:53:38 +00:00
parent a6d9264e17
commit 723ed9306a
2 changed files with 16 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
lib/Analysis/ModuleSummaryAnalysis.cpp
View File

@ -447,6 +447,12 @@ ModuleSummaryIndex llvm::buildModuleSummaryIndex(
auto &Summary = GlobalList.second[0];
bool AllRefsCanBeExternallyReferenced =
llvm::all_of(Summary->refs(), [&](const ValueInfo &VI) {
// If a global value definition references an unnamed global,
// be conservative. They're valid IR so we don't want to crash
// when we encounter any of them but they're infrequent enough
// that we don't bother optimizing them.
if (!VI.getValue()->hasName())
return false;
return !CantBePromoted.count(VI.getValue()->getGUID());
});
if (!AllRefsCanBeExternallyReferenced) {

10
test/Transforms/FunctionImport/unnamed-globals.ll Normal file
View File

@ -0,0 +1,10 @@
; Make sure we don't crash when referencing an unnamed global.
; RUN: opt %s -module-summary-analysis -S
@0 = external global [1 x { i64 }]
define internal void @tinkywinky() {
call void @patatino(i64 ptrtoint ([1 x { i64 }]* @0 to i64), i64 4)
ret void
}
declare void @patatino(i64, i64)