6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-07-25 13:24:46 +00:00
Code Issues Projects Releases Wiki Activity

[objc-arc] Remove the alias analysis part of r185764.

Browse Source 
Upon further reflection, the alias analysis part of r185764 is not a safe
change.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@185770 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Michael Gottesman
2013-07-07 04:18:03 +00:00
parent 0a80c12402
commit 95a1b3484d
2 changed files with 0 additions and 26 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
lib/Transforms/ObjCARC/ObjCARCAliasAnalysis.cpp
View File

@@ -150,14 +150,6 @@ ObjCARCAliasAnalysis::getModRefInfo(ImmutableCallSite CS, const Location &Loc) {
break;
}
// Handle special objective c calls defaulting to chaining.
const Function *F = CS.getCalledFunction();
if (F)
return StringSwitch<AliasAnalysis::ModRefResult>(F->getName())
.Case("objc_sync_start", NoModRef)
.Case("objc_sync_stop", NoModRef)
.Default(AliasAnalysis::getModRefInfo(CS, Loc));
return AliasAnalysis::getModRefInfo(CS, Loc);
}

18
test/Transforms/ObjCARC/gvn.ll
View File

@@ -3,8 +3,6 @@
@x = common global i8* null, align 8
declare i8* @objc_retain(i8*)
declare i32 @objc_sync_start(i8*)
declare i32 @objc_sync_stop(i8*)
; GVN should be able to eliminate this redundant load, with ARC-specific
; alias analysis.
@@ -22,19 +20,3 @@ entry:
%t = load i8** @x
ret i8* %t
}
; CHECK: define i8* @test1(i32 %n)
; CHECK-NEXT: entry:
; CHECK-NEXT: %s = load i8** @x
; CHECK-NEXT: call i32 @objc_sync_start
; CHECK-NEXT: call i32 @objc_sync_stop
; CHECK-NEXT: ret i8* %s
; CHECK-NEXT: }
define i8* @test1(i32 %n) nounwind {
entry:
%s = load i8** @x
%0 = call i32 @objc_sync_start(i8* %s)
%t = load i8** @x
%1 = call i32 @objc_sync_stop(i8* %s)
ret i8* %t
}