Skip to content

Commit 13fec08

Browse files
committed
remove HasLocalSummary var as the Summary doesn't get built in this patch
1 parent 5b686ad commit 13fec08

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

llvm/lib/Transforms/IPO/WholeProgramDevirt.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1363,10 +1363,10 @@ bool DevirtModule::trySingleImplDevirt(
13631363
if (!IsExported)
13641364
return false;
13651365

1366-
// If the only implementation has local linkage, we must promote
1367-
// to external to make it visible to thin LTO objects.
1368-
// This change should be safe only in LTO mode.
1369-
if (!HasLocalSummary && TheFn->hasLocalLinkage()) {
1366+
// If the only implementation has local linkage, we must promote to external
1367+
// to make it visible to thin LTO objects. We can only get here during the
1368+
// ThinLTO export phase.
1369+
if (TheFn->hasLocalLinkage()) {
13701370
std::string NewName = (TheFn->getName() + ".llvm.merged").str();
13711371

13721372
// Since we are renaming the function, any comdats with the same name must

0 commit comments

Comments
 (0)