RPCS3/llvm-mirror
1
0
Fork 0
mirror of https://github.com/RPCS3/llvm-mirror.git synced 2024-11-24 03:33:20 +01: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.

llvm-svn: 185770
This commit is contained in:
Michael Gottesman 2013-07-07 04:18:03 +00:00
parent 92317b03c4
commit 80ca18cc8d
2 changed files with 0 additions and 26 deletions
Show Stats 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
}