Skip to content

Commit 3bac3c9

Browse files
ahatanakacachemeifyoucan
authored andcommitted
Change the return type of UpgradeARCRuntimeCalls to void
Nothing is using the function return. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@368367 91177308-0d34-0410-b5e6-96231b3b80d8 (cherry picked from commit 33c8afb) apple-llvm-split-commit: 1c6215dea3c808404e055e6296833402fa2345d7 apple-llvm-split-dir: llvm/
1 parent ad21126 commit 3bac3c9

File tree

2 files changed

+6
-10
lines changed

2 files changed

+6
-10
lines changed

llvm/include/llvm/IR/AutoUpgrade.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ namespace llvm {
6161

6262
/// Convert calls to ARC runtime functions to intrinsic calls if the bitcode
6363
/// has the arm64 retainAutoreleasedReturnValue marker.
64-
bool UpgradeARCRuntimeCalls(Module &M);
64+
void UpgradeARCRuntimeCalls(Module &M);
6565

6666
void UpgradeSectionAttributes(Module &M);
6767

llvm/lib/IR/AutoUpgrade.cpp

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -3653,28 +3653,26 @@ bool llvm::UpgradeRetainReleaseMarker(Module &M) {
36533653
return Changed;
36543654
}
36553655

3656-
bool llvm::UpgradeARCRuntimeCalls(Module &M) {
3656+
void llvm::UpgradeARCRuntimeCalls(Module &M) {
36573657
auto UpgradeToIntrinsic = [&](const char *OldFunc,
36583658
llvm::Intrinsic::ID IntrinsicFunc) {
36593659
Function *Fn = M.getFunction(OldFunc);
36603660

36613661
if (!Fn)
3662-
return false;
3662+
return;
36633663

36643664
Function *NewFn = llvm::Intrinsic::getDeclaration(&M, IntrinsicFunc);
36653665
Fn->replaceAllUsesWith(NewFn);
36663666
Fn->eraseFromParent();
3667-
return true;
36683667
};
36693668

36703669
// Unconditionally convert "clang.arc.use" to "llvm.objc.clang.arc.use".
3671-
bool Changed =
3672-
UpgradeToIntrinsic("clang.arc.use", llvm::Intrinsic::objc_clang_arc_use);
3670+
UpgradeToIntrinsic("clang.arc.use", llvm::Intrinsic::objc_clang_arc_use);
36733671

36743672
// Return if the bitcode doesn't have the arm64 retainAutoreleasedReturnValue
36753673
// marker. We don't know for sure that it was compiled with ARC in that case.
36763674
if (!M.getModuleFlag("clang.arc.retainAutoreleasedReturnValueMarker"))
3677-
return false;
3675+
return;
36783676

36793677
std::pair<const char *, llvm::Intrinsic::ID> RuntimeFuncs[] = {
36803678
{"objc_autorelease", llvm::Intrinsic::objc_autorelease},
@@ -3716,9 +3714,7 @@ bool llvm::UpgradeARCRuntimeCalls(Module &M) {
37163714
llvm::Intrinsic::objc_arc_annotation_bottomup_bbend}};
37173715

37183716
for (auto &I : RuntimeFuncs)
3719-
Changed |= UpgradeToIntrinsic(I.first, I.second);
3720-
3721-
return Changed;
3717+
UpgradeToIntrinsic(I.first, I.second);
37223718
}
37233719

37243720
bool llvm::UpgradeModuleFlags(Module &M) {

0 commit comments

Comments
 (0)