[BasicAliasAnalysis] Take into account operand bundles in the getModRefInfo function

Differential Revision: http://reviews.llvm.org/D16225

llvm-svn: 257991
This commit is contained in:
Igor Laevsky 2016-01-16 12:15:53 +00:00
parent e05fcecd36
commit 28eeb3f66c
2 changed files with 35 additions and 4 deletions

View File

@ -717,14 +717,14 @@ ModRefInfo BasicAAResult::getModRefInfo(ImmutableCallSite CS,
if (!isa<Constant>(Object) && CS.getInstruction() != Object &&
isNonEscapingLocalObject(Object)) {
bool PassedAsArg = false;
unsigned ArgNo = 0;
for (ImmutableCallSite::arg_iterator CI = CS.arg_begin(), CE = CS.arg_end();
CI != CE; ++CI, ++ArgNo) {
unsigned OperandNo = 0;
for (auto CI = CS.data_operands_begin(), CE = CS.data_operands_end();
CI != CE; ++CI, ++OperandNo) {
// Only look at the no-capture or byval pointer arguments. If this
// pointer were passed to arguments that were neither of these, then it
// couldn't be no-capture.
if (!(*CI)->getType()->isPointerTy() ||
(!CS.doesNotCapture(ArgNo) && !CS.isByValArgument(ArgNo)))
(!CS.doesNotCapture(OperandNo) && !CS.isByValArgument(OperandNo)))
continue;
// If this is a no-capture pointer argument, see if we can tell that it

View File

@ -0,0 +1,31 @@
; RUN: opt < %s -basicaa -dse -S | FileCheck %s
declare noalias i8* @malloc(i64) "malloc-like"
declare void @foo()
declare void @bar(i8*)
define void @test() {
%obj = call i8* @malloc(i64 8)
store i8 0, i8* %obj
; don't remove store. %obj should be treated like it will be read by the @foo.
; CHECK: store i8 0, i8* %obj
call void @foo() ["deopt" (i8* %obj)]
ret void
}
define void @test1() {
%obj = call i8* @malloc(i64 8)
store i8 0, i8* %obj
; CHECK: store i8 0, i8* %obj
call void @bar(i8* nocapture %obj)
ret void
}
define void @test2() {
%obj = call i8* @malloc(i64 8)
store i8 0, i8* %obj
; CHECK-NOT: store i8 0, i8* %obj
call void @foo()
ret void
}