diff --git a/llvm/lib/Transforms/ObjCARC/ObjCARCContract.cpp b/llvm/lib/Transforms/ObjCARC/ObjCARCContract.cpp index e190c46b613b0a..c2499fc85cf52d 100644 --- a/llvm/lib/Transforms/ObjCARC/ObjCARCContract.cpp +++ b/llvm/lib/Transforms/ObjCARC/ObjCARCContract.cpp @@ -476,7 +476,7 @@ bool ObjCARCContract::tryToPeepholeInstruction( --BBI; } while (IsNoopInstruction(&*BBI)); - if (&*BBI == GetArgRCIdentityRoot(Inst)) { + if (GetRCIdentityRoot(&*BBI) == GetArgRCIdentityRoot(Inst)) { LLVM_DEBUG(dbgs() << "Adding inline asm marker for the return value " "optimization.\n"); Changed = true; diff --git a/llvm/test/Transforms/ObjCARC/contract-marker.ll b/llvm/test/Transforms/ObjCARC/contract-marker.ll index a93bbe3a290129..e2217068d5d1ca 100644 --- a/llvm/test/Transforms/ObjCARC/contract-marker.ll +++ b/llvm/test/Transforms/ObjCARC/contract-marker.ll @@ -32,11 +32,31 @@ entry: ret void } +; CHECK-LABEL: define i8* @foo3( +; CHECK: call i8* @returnsArg( +; CHECK-NEXT: call void asm sideeffect + +define i8* @foo3(i8* %a) { + %call = call i8* @returnsArg(i8* %a) + call i8* @llvm.objc.retainAutoreleasedReturnValue(i8* %call) + ret i8* %call +} + +; CHECK-LABEL: define i8* @foo4( +; CHECK: call i8* @returnsArg( +; CHECK-NEXT: call void asm sideeffect + +define i8* @foo4(i8* %a) { + %call = call i8* @returnsArg(i8* %a) + call i8* @llvm.objc.retainAutoreleasedReturnValue(i8* %a) + ret i8* %call +} declare i32* @qux() declare i8* @llvm.objc.retainAutoreleasedReturnValue(i8*) declare i8* @llvm.objc.unsafeClaimAutoreleasedReturnValue(i8*) declare void @bar(i8*) +declare i8* @returnsArg(i8* returned) !llvm.module.flags = !{!0}