Add nonlazybind to objc_retain/objc_release when converting from intrinsics.
For performance reasons, clang set nonlazybind on these functions. Now that we
are using intrinsics instead of runtime calls, we should set this attribute when
creating the runtime functions.
llvm-svn: 349558
diff --git a/llvm/lib/CodeGen/PreISelIntrinsicLowering.cpp b/llvm/lib/CodeGen/PreISelIntrinsicLowering.cpp
index cd15f6b..3689dab 100644
--- a/llvm/lib/CodeGen/PreISelIntrinsicLowering.cpp
+++ b/llvm/lib/CodeGen/PreISelIntrinsicLowering.cpp
@@ -57,7 +57,8 @@
return Changed;
}
-static bool lowerObjCCall(Function &F, const char *NewFn) {
+static bool lowerObjCCall(Function &F, const char *NewFn,
+ bool setNonLazyBind = false) {
if (F.use_empty())
return false;
@@ -65,6 +66,12 @@
// program already contains a function with this name.
Module *M = F.getParent();
Constant* FCache = M->getOrInsertFunction(NewFn, F.getFunctionType());
+
+ // If we have Native ARC, set nonlazybind attribute for these APIs for
+ // performance.
+ if (setNonLazyBind)
+ if (Function* Fn = dyn_cast<Function>(FCache))
+ Fn->addFnAttr(Attribute::NonLazyBind);
for (auto I = F.use_begin(), E = F.use_end(); I != E;) {
auto *CI = dyn_cast<CallInst>(I->getUser());
@@ -125,10 +132,10 @@
Changed |= lowerObjCCall(F, "objc_moveWeak");
break;
case Intrinsic::objc_release:
- Changed |= lowerObjCCall(F, "objc_release");
+ Changed |= lowerObjCCall(F, "objc_release", true);
break;
case Intrinsic::objc_retain:
- Changed |= lowerObjCCall(F, "objc_retain");
+ Changed |= lowerObjCCall(F, "objc_retain", true);
break;
case Intrinsic::objc_retainAutorelease:
Changed |= lowerObjCCall(F, "objc_retainAutorelease");