[llvm] r373986 - [Attributor] Use local linkage instead of internal
Johannes Doerfert via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 7 16:21:52 PDT 2019
Author: jdoerfert
Date: Mon Oct 7 16:21:52 2019
New Revision: 373986
URL: http://llvm.org/viewvc/llvm-project?rev=373986&view=rev
Log:
[Attributor] Use local linkage instead of internal
Local linkage is internal or private, and private is a specialization of
internal, so either is fine for all our "local linkage" queries.
Modified:
llvm/trunk/include/llvm/Transforms/IPO/Attributor.h
llvm/trunk/lib/Transforms/IPO/Attributor.cpp
llvm/trunk/test/Transforms/FunctionAttrs/internal-noalias.ll
Modified: llvm/trunk/include/llvm/Transforms/IPO/Attributor.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Transforms/IPO/Attributor.h?rev=373986&r1=373985&r2=373986&view=diff
==============================================================================
--- llvm/trunk/include/llvm/Transforms/IPO/Attributor.h (original)
+++ llvm/trunk/include/llvm/Transforms/IPO/Attributor.h Mon Oct 7 16:21:52 2019
@@ -810,8 +810,8 @@ struct Attributor {
/// This will trigger the identification and initialization of attributes for
/// \p F.
void markLiveInternalFunction(const Function &F) {
- assert(F.hasInternalLinkage() &&
- "Only internal linkage is assumed dead initially.");
+ assert(F.hasLocalLinkage() &&
+ "Only local linkage is assumed dead initially.");
identifyDefaultAbstractAttributes(const_cast<Function &>(F));
}
Modified: llvm/trunk/lib/Transforms/IPO/Attributor.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/IPO/Attributor.cpp?rev=373986&r1=373985&r2=373986&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/IPO/Attributor.cpp (original)
+++ llvm/trunk/lib/Transforms/IPO/Attributor.cpp Mon Oct 7 16:21:52 2019
@@ -2081,7 +2081,7 @@ struct AAIsDeadImpl : public AAIsDead {
for (const Instruction &I : BB)
if (ImmutableCallSite ICS = ImmutableCallSite(&I))
if (const Function *F = ICS.getCalledFunction())
- if (F->hasInternalLinkage())
+ if (F->hasLocalLinkage())
A.markLiveInternalFunction(*F);
}
@@ -3933,7 +3933,7 @@ bool Attributor::checkForAllCallSites(
if (!AssociatedFunction)
return false;
- if (RequireAllCallSites && !AssociatedFunction->hasInternalLinkage()) {
+ if (RequireAllCallSites && !AssociatedFunction->hasLocalLinkage()) {
LLVM_DEBUG(
dbgs()
<< "[Attributor] Function " << AssociatedFunction->getName()
@@ -4319,7 +4319,7 @@ ChangeStatus Attributor::run(Module &M)
// below fixpoint loop will identify and eliminate them.
SmallVector<Function *, 8> InternalFns;
for (Function &F : M)
- if (F.hasInternalLinkage())
+ if (F.hasLocalLinkage())
InternalFns.push_back(&F);
bool FoundDeadFn = true;
@@ -4634,7 +4634,7 @@ static bool runAttributorOnModule(Module
// We look at internal functions only on-demand but if any use is not a
// direct call, we have to do it eagerly.
- if (F.hasInternalLinkage()) {
+ if (F.hasLocalLinkage()) {
if (llvm::all_of(F.uses(), [](const Use &U) {
return ImmutableCallSite(U.getUser()) &&
ImmutableCallSite(U.getUser()).isCallee(&U);
Modified: llvm/trunk/test/Transforms/FunctionAttrs/internal-noalias.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Transforms/FunctionAttrs/internal-noalias.ll?rev=373986&r1=373985&r2=373986&view=diff
==============================================================================
--- llvm/trunk/test/Transforms/FunctionAttrs/internal-noalias.ll (original)
+++ llvm/trunk/test/Transforms/FunctionAttrs/internal-noalias.ll Mon Oct 7 16:21:52 2019
@@ -8,9 +8,9 @@ entry:
ret i32 %add
}
-; CHECK: define internal i32 @noalias_args(i32* nocapture readonly %A, i32* noalias nocapture readonly %B)
+; CHECK: define private i32 @noalias_args(i32* nocapture readonly %A, i32* noalias nocapture readonly %B)
-define internal i32 @noalias_args(i32* %A, i32* %B) #0 {
+define private i32 @noalias_args(i32* %A, i32* %B) #0 {
entry:
%0 = load i32, i32* %A, align 4
%1 = load i32, i32* %B, align 4
More information about the llvm-commits
mailing list