X-Git-Url: http://demsky.eecs.uci.edu/git/?a=blobdiff_plain;f=lib%2FAnalysis%2FBasicAliasAnalysis.cpp;h=fe71f04098b13061a0b04c4dcf2a99bc3fd7f9dd;hb=002ec1482c83003c6aef841395271aa5993d916a;hp=89b5d5c3cb9000fe6333ccdc26a2d99ea808341d;hpb=ef1502925851c813a57bb2257be3026e9de1fdd1;p=oota-llvm.git diff --git a/lib/Analysis/BasicAliasAnalysis.cpp b/lib/Analysis/BasicAliasAnalysis.cpp index 89b5d5c3cb9..fe71f04098b 100644 --- a/lib/Analysis/BasicAliasAnalysis.cpp +++ b/lib/Analysis/BasicAliasAnalysis.cpp @@ -2,8 +2,8 @@ // // The LLVM Compiler Infrastructure // -// This file was developed by the LLVM research group and is distributed under -// the University of Illinois Open Source License. See LICENSE.TXT for details. +// This file is distributed under the University of Illinois Open Source +// License. See LICENSE.TXT for details. // //===----------------------------------------------------------------------===// // @@ -14,19 +14,17 @@ //===----------------------------------------------------------------------===// #include "llvm/Analysis/AliasAnalysis.h" +#include "llvm/Analysis/CaptureTracking.h" #include "llvm/Analysis/Passes.h" #include "llvm/Constants.h" #include "llvm/DerivedTypes.h" #include "llvm/Function.h" -#include "llvm/ParameterAttributes.h" #include "llvm/GlobalVariable.h" #include "llvm/Instructions.h" -#include "llvm/Intrinsics.h" +#include "llvm/IntrinsicInst.h" #include "llvm/Pass.h" #include "llvm/Target/TargetData.h" #include "llvm/ADT/SmallVector.h" -#include "llvm/ADT/StringMap.h" -#include "llvm/ADT/BitVector.h" #include "llvm/ADT/STLExtras.h" #include "llvm/Support/Compiler.h" #include "llvm/Support/GetElementPtrTypeIterator.h" @@ -34,6 +32,105 @@ #include using namespace llvm; +//===----------------------------------------------------------------------===// +// Useful predicates +//===----------------------------------------------------------------------===// + +static const User *isGEP(const Value *V) { + if (isa(V) || + (isa(V) && + cast(V)->getOpcode() == Instruction::GetElementPtr)) + return cast(V); + return 0; +} + +static const Value *GetGEPOperands(const Value *V, + SmallVector &GEPOps) { + assert(GEPOps.empty() && "Expect empty list to populate!"); + GEPOps.insert(GEPOps.end(), cast(V)->op_begin()+1, + cast(V)->op_end()); + + // Accumulate all of the chained indexes into the operand array + V = cast(V)->getOperand(0); + + while (const User *G = isGEP(V)) { + if (!isa(GEPOps[0]) || isa(GEPOps[0]) || + !cast(GEPOps[0])->isNullValue()) + break; // Don't handle folding arbitrary pointer offsets yet... + GEPOps.erase(GEPOps.begin()); // Drop the zero index + GEPOps.insert(GEPOps.begin(), G->op_begin()+1, G->op_end()); + V = G->getOperand(0); + } + return V; +} + +/// isKnownNonNull - Return true if we know that the specified value is never +/// null. +static bool isKnownNonNull(const Value *V) { + // Alloca never returns null, malloc might. + if (isa(V)) return true; + + // A byval argument is never null. + if (const Argument *A = dyn_cast(V)) + return A->hasByValAttr(); + + // Global values are not null unless extern weak. + if (const GlobalValue *GV = dyn_cast(V)) + return !GV->hasExternalWeakLinkage(); + return false; +} + +/// isNonEscapingLocalObject - Return true if the pointer is to a function-local +/// object that never escapes from the function. +static bool isNonEscapingLocalObject(const Value *V) { + // If this is a local allocation, check to see if it escapes. + if (isa(V) || isNoAliasCall(V)) + return !PointerMayBeCaptured(V, false); + + // If this is an argument that corresponds to a byval or noalias argument, + // then it has not escaped before entering the function. Check if it escapes + // inside the function. + if (const Argument *A = dyn_cast(V)) + if (A->hasByValAttr() || A->hasNoAliasAttr()) { + // Don't bother analyzing arguments already known not to escape. + if (A->hasNoCaptureAttr()) + return true; + return !PointerMayBeCaptured(V, false); + } + return false; +} + + +/// isObjectSmallerThan - Return true if we can prove that the object specified +/// by V is smaller than Size. +static bool isObjectSmallerThan(const Value *V, unsigned Size, + const TargetData &TD) { + const Type *AccessTy; + if (const GlobalVariable *GV = dyn_cast(V)) { + AccessTy = GV->getType()->getElementType(); + } else if (const AllocationInst *AI = dyn_cast(V)) { + if (!AI->isArrayAllocation()) + AccessTy = AI->getType()->getElementType(); + else + return false; + } else if (const Argument *A = dyn_cast(V)) { + if (A->hasByValAttr()) + AccessTy = cast(A->getType())->getElementType(); + else + return false; + } else { + return false; + } + + if (AccessTy->isSized()) + return TD.getTypePaddedSize(AccessTy) < Size; + return false; +} + +//===----------------------------------------------------------------------===// +// NoAA Pass +//===----------------------------------------------------------------------===// + namespace { /// NoAA - This class implements the -no-aa pass, which always returns "I /// don't know" for alias queries. NoAA is unlike other alias analysis @@ -42,8 +139,8 @@ namespace { /// struct VISIBILITY_HIDDEN NoAA : public ImmutablePass, public AliasAnalysis { static char ID; // Class identification, replacement for typeinfo - NoAA() : ImmutablePass((intptr_t)&ID) {} - explicit NoAA(intptr_t PID) : ImmutablePass(PID) { } + NoAA() : ImmutablePass(&ID) {} + explicit NoAA(void *PID) : ImmutablePass(PID) { } virtual void getAnalysisUsage(AnalysisUsage &AU) const { AU.addRequired(); @@ -58,11 +155,6 @@ namespace { return MayAlias; } - virtual ModRefBehavior getModRefBehavior(Function *F, CallSite CS, - std::vector *Info) { - return UnknownModRefBehavior; - } - virtual void getArgumentAccesses(Function *F, CallSite CS, std::vector &Info) { assert(0 && "This method may not be called on this function!"); @@ -81,32 +173,34 @@ namespace { virtual void deleteValue(Value *V) {} virtual void copyValue(Value *From, Value *To) {} }; +} // End of anonymous namespace - // Register this pass... - char NoAA::ID = 0; - RegisterPass - U("no-aa", "No Alias Analysis (always returns 'may' alias)"); +// Register this pass... +char NoAA::ID = 0; +static RegisterPass +U("no-aa", "No Alias Analysis (always returns 'may' alias)", true, true); - // Declare that we implement the AliasAnalysis interface - RegisterAnalysisGroup V(U); -} // End of anonymous namespace +// Declare that we implement the AliasAnalysis interface +static RegisterAnalysisGroup V(U); ImmutablePass *llvm::createNoAAPass() { return new NoAA(); } +//===----------------------------------------------------------------------===// +// BasicAA Pass +//===----------------------------------------------------------------------===// + namespace { /// BasicAliasAnalysis - This is the default alias analysis implementation. /// Because it doesn't chain to a previous alias analysis (like -no-aa), it /// derives from the NoAA class. struct VISIBILITY_HIDDEN BasicAliasAnalysis : public NoAA { static char ID; // Class identification, replacement for typeinfo - BasicAliasAnalysis() : NoAA((intptr_t)&ID) { } + BasicAliasAnalysis() : NoAA(&ID) {} AliasResult alias(const Value *V1, unsigned V1Size, const Value *V2, unsigned V2Size); ModRefResult getModRefInfo(CallSite CS, Value *P, unsigned Size); - ModRefResult getModRefInfo(CallSite CS1, CallSite CS2) { - return NoAA::getModRefInfo(CS1,CS2); - } + ModRefResult getModRefInfo(CallSite CS1, CallSite CS2); /// hasNoModRefInfoForCalls - We can provide mod/ref information against /// non-escaping allocations. @@ -116,9 +210,6 @@ namespace { /// global) or not. bool pointsToConstantMemory(const Value *P); - virtual ModRefBehavior getModRefBehavior(Function *F, CallSite CS, - std::vector *Info); - private: // CheckGEPInstructions - Check two GEP instructions with known // must-aliasing base pointers. This checks to see if the index expressions @@ -129,115 +220,30 @@ namespace { const Type *BasePtr2Ty, Value **GEP2Ops, unsigned NumGEP2Ops, unsigned G2Size); }; +} // End of anonymous namespace - // Register this pass... - char BasicAliasAnalysis::ID = 0; - RegisterPass - X("basicaa", "Basic Alias Analysis (default AA impl)"); +// Register this pass... +char BasicAliasAnalysis::ID = 0; +static RegisterPass +X("basicaa", "Basic Alias Analysis (default AA impl)", false, true); - // Declare that we implement the AliasAnalysis interface - RegisterAnalysisGroup Y(X); -} // End of anonymous namespace +// Declare that we implement the AliasAnalysis interface +static RegisterAnalysisGroup Y(X); ImmutablePass *llvm::createBasicAliasAnalysisPass() { return new BasicAliasAnalysis(); } -// getUnderlyingObject - This traverses the use chain to figure out what object -// the specified value points to. If the value points to, or is derived from, a -// unique object or an argument, return it. -static const Value *getUnderlyingObject(const Value *V) { - if (!isa(V->getType())) return 0; - - // If we are at some type of object, return it. GlobalValues and Allocations - // have unique addresses. - if (isa(V) || isa(V) || isa(V)) - return V; - - // Traverse through different addressing mechanisms... - if (const Instruction *I = dyn_cast(V)) { - if (isa(I) || isa(I)) - return getUnderlyingObject(I->getOperand(0)); - } else if (const ConstantExpr *CE = dyn_cast(V)) { - if (CE->getOpcode() == Instruction::BitCast || - CE->getOpcode() == Instruction::GetElementPtr) - return getUnderlyingObject(CE->getOperand(0)); - } - return 0; -} - -static const User *isGEP(const Value *V) { - if (isa(V) || - (isa(V) && - cast(V)->getOpcode() == Instruction::GetElementPtr)) - return cast(V); - return 0; -} - -static const Value *GetGEPOperands(const Value *V, - SmallVector &GEPOps){ - assert(GEPOps.empty() && "Expect empty list to populate!"); - GEPOps.insert(GEPOps.end(), cast(V)->op_begin()+1, - cast(V)->op_end()); - - // Accumulate all of the chained indexes into the operand array - V = cast(V)->getOperand(0); - - while (const User *G = isGEP(V)) { - if (!isa(GEPOps[0]) || isa(GEPOps[0]) || - !cast(GEPOps[0])->isNullValue()) - break; // Don't handle folding arbitrary pointer offsets yet... - GEPOps.erase(GEPOps.begin()); // Drop the zero index - GEPOps.insert(GEPOps.begin(), G->op_begin()+1, G->op_end()); - V = G->getOperand(0); - } - return V; -} /// pointsToConstantMemory - Chase pointers until we find a (constant /// global) or not. bool BasicAliasAnalysis::pointsToConstantMemory(const Value *P) { - if (const Value *V = getUnderlyingObject(P)) - if (const GlobalVariable *GV = dyn_cast(V)) - return GV->isConstant(); + if (const GlobalVariable *GV = + dyn_cast(P->getUnderlyingObject())) + return GV->isConstant(); return false; } -// Determine if an AllocationInst instruction escapes from the function it is -// contained in. If it does not escape, there is no way for another function to -// mod/ref it. We do this by looking at its uses and determining if the uses -// can escape (recursively). -static bool AddressMightEscape(const Value *V) { - for (Value::use_const_iterator UI = V->use_begin(), E = V->use_end(); - UI != E; ++UI) { - const Instruction *I = cast(*UI); - switch (I->getOpcode()) { - case Instruction::Load: - break; //next use. - case Instruction::Store: - if (I->getOperand(0) == V) - return true; // Escapes if the pointer is stored. - break; // next use. - case Instruction::GetElementPtr: - if (AddressMightEscape(I)) - return true; - break; // next use. - case Instruction::BitCast: - if (!isa(I->getType())) - return true; - if (AddressMightEscape(I)) - return true; - break; // next use - case Instruction::Ret: - // If returned, the address will escape to calling functions, but no - // callees could modify it. - break; // next use - default: - return true; - } - } - return false; -} // getModRefInfo - Check to see if the specified callsite can clobber the // specified memory object. Since we only look at local properties of this @@ -246,44 +252,61 @@ static bool AddressMightEscape(const Value *V) { // AliasAnalysis::ModRefResult BasicAliasAnalysis::getModRefInfo(CallSite CS, Value *P, unsigned Size) { - if (!isa(P)) - if (const AllocationInst *AI = - dyn_cast_or_null(getUnderlyingObject(P))) { - // Okay, the pointer is to a stack allocated object. If we can prove that - // the pointer never "escapes", then we know the call cannot clobber it, - // because it simply can't get its address. - if (!AddressMightEscape(AI)) - return NoModRef; - - // If this is a tail call and P points to a stack location, we know that - // the tail call cannot access or modify the local stack. + if (!isa(P)) { + const Value *Object = P->getUnderlyingObject(); + + // If this is a tail call and P points to a stack location, we know that + // the tail call cannot access or modify the local stack. + // We cannot exclude byval arguments here; these belong to the caller of + // the current function not to the current function, and a tail callee + // may reference them. + if (isa(Object)) if (CallInst *CI = dyn_cast(CS.getInstruction())) - if (CI->isTailCall() && isa(AI)) + if (CI->isTailCall()) return NoModRef; + + // If the pointer is to a locally allocated object that does not escape, + // then the call can not mod/ref the pointer unless the call takes the + // argument without capturing it. + if (isNonEscapingLocalObject(Object) && CS.getInstruction() != Object) { + bool passedAsArg = false; + // TODO: Eventually only check 'nocapture' arguments. + for (CallSite::arg_iterator CI = CS.arg_begin(), CE = CS.arg_end(); + CI != CE; ++CI) + if (isa((*CI)->getType()) && + alias(cast(CI), ~0U, P, ~0U) != NoAlias) + passedAsArg = true; + + if (!passedAsArg) + return NoModRef; } + } // The AliasAnalysis base class has some smarts, lets use them. return AliasAnalysis::getModRefInfo(CS, P, Size); } -static bool isNoAliasArgument(const Argument *Arg) { - const Function *Func = Arg->getParent(); - const ParamAttrsList *Attr = Func->getFunctionType()->getParamAttrs(); - if (Attr) { - unsigned Idx = 1; - for (Function::const_arg_iterator I = Func->arg_begin(), - E = Func->arg_end(); I != E; ++I, ++Idx) { - if (&(*I) == Arg && - Attr->paramHasAttr(Idx, ParamAttr::NoAlias)) - return true; - } - } - return false; + +AliasAnalysis::ModRefResult +BasicAliasAnalysis::getModRefInfo(CallSite CS1, CallSite CS2) { + // If CS1 or CS2 are readnone, they don't interact. + ModRefBehavior CS1B = AliasAnalysis::getModRefBehavior(CS1); + if (CS1B == DoesNotAccessMemory) return NoModRef; + + ModRefBehavior CS2B = AliasAnalysis::getModRefBehavior(CS2); + if (CS2B == DoesNotAccessMemory) return NoModRef; + + // If they both only read from memory, just return ref. + if (CS1B == OnlyReadsMemory && CS2B == OnlyReadsMemory) + return Ref; + + // Otherwise, fall back to NoAA (mod+ref). + return NoAA::getModRefInfo(CS1, CS2); } + // alias - Provide a bunch of ad-hoc rules to disambiguate in common cases, such -// as array references. Note that this function is heavily tail recursive. -// Hopefully we have a smart C++ compiler. :) +// as array references. // AliasAnalysis::AliasResult BasicAliasAnalysis::alias(const Value *V1, unsigned V1Size, @@ -299,114 +322,88 @@ BasicAliasAnalysis::alias(const Value *V1, unsigned V1Size, // Are we checking for alias of the same value? if (V1 == V2) return MustAlias; - if ((!isa(V1->getType()) || !isa(V2->getType())) && - V1->getType() != Type::Int64Ty && V2->getType() != Type::Int64Ty) + if (!isa(V1->getType()) || !isa(V2->getType())) return NoAlias; // Scalars cannot alias each other - // Strip off cast instructions... + // Strip off cast instructions. Since V1 and V2 are pointers, they must be + // pointer<->pointer bitcasts. if (const BitCastInst *I = dyn_cast(V1)) return alias(I->getOperand(0), V1Size, V2, V2Size); if (const BitCastInst *I = dyn_cast(V2)) return alias(V1, V1Size, I->getOperand(0), V2Size); - // Figure out what objects these things are pointing to if we can... - const Value *O1 = getUnderlyingObject(V1); - const Value *O2 = getUnderlyingObject(V2); - - // Pointing at a discernible object? - if (O1) { - if (O2) { - if (const Argument *O1Arg = dyn_cast(O1)) { - // Incoming argument cannot alias locally allocated object! - if (isa(O2)) return NoAlias; - - // If they are two different objects, and one is a noalias argument - // then they do not alias. - if (O1 != O2 && isNoAliasArgument(O1Arg)) - return NoAlias; - - // Otherwise, nothing is known... - } - - if (const Argument *O2Arg = dyn_cast(O2)) { - // Incoming argument cannot alias locally allocated object! - if (isa(O1)) return NoAlias; - - // If they are two different objects, and one is a noalias argument - // then they do not alias. - if (O1 != O2 && isNoAliasArgument(O2Arg)) - return NoAlias; - - // Otherwise, nothing is known... - - } else if (O1 != O2) { - if (!isa(O1)) - // If they are two different objects, and neither is an argument, - // we know that we have no alias... - return NoAlias; - } + // Figure out what objects these things are pointing to if we can. + const Value *O1 = V1->getUnderlyingObject(); + const Value *O2 = V2->getUnderlyingObject(); - // If they are the same object, they we can look at the indexes. If they - // index off of the object is the same for both pointers, they must alias. - // If they are provably different, they must not alias. Otherwise, we - // can't tell anything. - } - - - if (!isa(O1) && isa(V2)) - return NoAlias; // Unique values don't alias null - - if (isa(O1) || - (isa(O1) && - !cast(O1)->isArrayAllocation())) - if (cast(O1->getType())->getElementType()->isSized()) { - // If the size of the other access is larger than the total size of the - // global/alloca/malloc, it cannot be accessing the global (it's - // undefined to load or store bytes before or after an object). - const Type *ElTy = cast(O1->getType())->getElementType(); - unsigned GlobalSize = getTargetData().getTypeSize(ElTy); - if (GlobalSize < V2Size && V2Size != ~0U) - return NoAlias; - } - } + if (O1 != O2) { + // If V1/V2 point to two different objects we know that we have no alias. + if (isIdentifiedObject(O1) && isIdentifiedObject(O2)) + return NoAlias; + + // Arguments can't alias with local allocations or noalias calls. + if ((isa(O1) && (isa(O2) || isNoAliasCall(O2))) || + (isa(O2) && (isa(O1) || isNoAliasCall(O1)))) + return NoAlias; - if (O2) { - if (!isa(O2) && isa(V1)) - return NoAlias; // Unique values don't alias null - - if (isa(O2) || - (isa(O2) && - !cast(O2)->isArrayAllocation())) - if (cast(O2->getType())->getElementType()->isSized()) { - // If the size of the other access is larger than the total size of the - // global/alloca/malloc, it cannot be accessing the object (it's - // undefined to load or store bytes before or after an object). - const Type *ElTy = cast(O2->getType())->getElementType(); - unsigned GlobalSize = getTargetData().getTypeSize(ElTy); - if (GlobalSize < V1Size && V1Size != ~0U) - return NoAlias; - } + // Most objects can't alias null. + if ((isa(V2) && isKnownNonNull(O1)) || + (isa(V1) && isKnownNonNull(O2))) + return NoAlias; } - + + // If the size of one access is larger than the entire object on the other + // side, then we know such behavior is undefined and can assume no alias. + const TargetData &TD = getTargetData(); + if ((V1Size != ~0U && isObjectSmallerThan(O2, V1Size, TD)) || + (V2Size != ~0U && isObjectSmallerThan(O1, V2Size, TD))) + return NoAlias; + + // If one pointer is the result of a call/invoke and the other is a + // non-escaping local object, then we know the object couldn't escape to a + // point where the call could return it. + if ((isa(O1) || isa(O1)) && + isNonEscapingLocalObject(O2) && O1 != O2) + return NoAlias; + if ((isa(O2) || isa(O2)) && + isNonEscapingLocalObject(O1) && O1 != O2) + return NoAlias; + // If we have two gep instructions with must-alias'ing base pointers, figure // out if the indexes to the GEP tell us anything about the derived pointer. // Note that we also handle chains of getelementptr instructions as well as // constant expression getelementptrs here. // if (isGEP(V1) && isGEP(V2)) { + const User *GEP1 = cast(V1); + const User *GEP2 = cast(V2); + + // If V1 and V2 are identical GEPs, just recurse down on both of them. + // This allows us to analyze things like: + // P = gep A, 0, i, 1 + // Q = gep B, 0, i, 1 + // by just analyzing A and B. This is even safe for variable indices. + if (GEP1->getType() == GEP2->getType() && + GEP1->getNumOperands() == GEP2->getNumOperands() && + GEP1->getOperand(0)->getType() == GEP2->getOperand(0)->getType() && + // All operands are the same, ignoring the base. + std::equal(GEP1->op_begin()+1, GEP1->op_end(), GEP2->op_begin()+1)) + return alias(GEP1->getOperand(0), V1Size, GEP2->getOperand(0), V2Size); + + // Drill down into the first non-gep value, to test for must-aliasing of // the base pointers. - const Value *BasePtr1 = V1, *BasePtr2 = V2; - do { - BasePtr1 = cast(BasePtr1)->getOperand(0); - } while (isGEP(BasePtr1) && - cast(BasePtr1)->getOperand(1) == - Constant::getNullValue(cast(BasePtr1)->getOperand(1)->getType())); - do { - BasePtr2 = cast(BasePtr2)->getOperand(0); - } while (isGEP(BasePtr2) && - cast(BasePtr2)->getOperand(1) == - Constant::getNullValue(cast(BasePtr2)->getOperand(1)->getType())); + while (isGEP(GEP1->getOperand(0)) && + GEP1->getOperand(1) == + Constant::getNullValue(GEP1->getOperand(1)->getType())) + GEP1 = cast(GEP1->getOperand(0)); + const Value *BasePtr1 = GEP1->getOperand(0); + + while (isGEP(GEP2->getOperand(0)) && + GEP2->getOperand(1) == + Constant::getNullValue(GEP2->getOperand(1)->getType())) + GEP2 = cast(GEP2->getOperand(0)); + const Value *BasePtr2 = GEP2->getOperand(0); // Do the base pointers alias? AliasResult BaseAlias = alias(BasePtr1, ~0U, BasePtr2, ~0U); @@ -500,7 +497,7 @@ BasicAliasAnalysis::alias(const Value *V1, unsigned V1Size, return MayAlias; } -// This function is used to determin if the indices of two GEP instructions are +// This function is used to determine if the indices of two GEP instructions are // equal. V1 and V2 are the indices. static bool IndexOperandsEqual(Value *V1, Value *V2) { if (V1->getType() == V2->getType()) @@ -668,7 +665,7 @@ BasicAliasAnalysis::CheckGEPInstructions( if (isa(GEP1Ops[i]) && !cast(GEP1Ops[i])->isZero()) { // Yup, there's a constant in the tail. Set all variables to - // constants in the GEP instruction to make it suiteable for + // constants in the GEP instruction to make it suitable for // TargetData::getIndexedOffset. for (i = 0; i != MaxOperands; ++i) if (!isa(GEP1Ops[i])) @@ -683,9 +680,15 @@ BasicAliasAnalysis::CheckGEPInstructions( int64_t Offset2 = TD.getIndexedOffset(GEPPointerTy, GEP1Ops, MinOperands); + // Make sure we compare the absolute difference. + if (Offset1 > Offset2) + std::swap(Offset1, Offset2); + // If the tail provided a bit enough offset, return noalias! if ((uint64_t)(Offset2-Offset1) >= SizeMax) return NoAlias; + // Otherwise break - we don't look for another constant in the tail. + break; } } @@ -735,8 +738,8 @@ BasicAliasAnalysis::CheckGEPInstructions( if (const ArrayType *AT = dyn_cast(BasePtr1Ty)) { if (Op1C->getZExtValue() >= AT->getNumElements()) return MayAlias; // Be conservative with out-of-range accesses - } else if (const VectorType *PT = dyn_cast(BasePtr1Ty)) { - if (Op1C->getZExtValue() >= PT->getNumElements()) + } else if (const VectorType *VT = dyn_cast(BasePtr1Ty)) { + if (Op1C->getZExtValue() >= VT->getNumElements()) return MayAlias; // Be conservative with out-of-range accesses } @@ -753,20 +756,19 @@ BasicAliasAnalysis::CheckGEPInstructions( // if (const ArrayType *AT = dyn_cast(BasePtr1Ty)) GEP1Ops[i] = ConstantInt::get(Type::Int64Ty,AT->getNumElements()-1); - else if (const VectorType *PT = dyn_cast(BasePtr1Ty)) - GEP1Ops[i] = ConstantInt::get(Type::Int64Ty,PT->getNumElements()-1); - + else if (const VectorType *VT = dyn_cast(BasePtr1Ty)) + GEP1Ops[i] = ConstantInt::get(Type::Int64Ty,VT->getNumElements()-1); } } if (Op2) { if (const ConstantInt *Op2C = dyn_cast(Op2)) { // If this is an array index, make sure the array element is in range. - if (const ArrayType *AT = dyn_cast(BasePtr1Ty)) { + if (const ArrayType *AT = dyn_cast(BasePtr2Ty)) { if (Op2C->getZExtValue() >= AT->getNumElements()) return MayAlias; // Be conservative with out-of-range accesses - } else if (const VectorType *PT = dyn_cast(BasePtr1Ty)) { - if (Op2C->getZExtValue() >= PT->getNumElements()) + } else if (const VectorType *VT = dyn_cast(BasePtr2Ty)) { + if (Op2C->getZExtValue() >= VT->getNumElements()) return MayAlias; // Be conservative with out-of-range accesses } } else { // Conservatively assume the minimum value for this index @@ -795,8 +797,13 @@ BasicAliasAnalysis::CheckGEPInstructions( getTargetData().getIndexedOffset(GEPPointerTy, GEP1Ops, NumGEP1Ops); int64_t Offset2 = getTargetData().getIndexedOffset(GEPPointerTy, GEP2Ops, NumGEP2Ops); - assert(Offset1 Offset2) + std::swap(Offset1, Offset2); + if ((uint64_t)(Offset2-Offset1) >= SizeMax) { //cerr << "Determined that these two GEP's don't alias [" // << SizeMax << " bytes]: \n" << *GEP1 << *GEP2; @@ -806,147 +813,5 @@ BasicAliasAnalysis::CheckGEPInstructions( return MayAlias; } -namespace { - struct VISIBILITY_HIDDEN StringCompare { - bool operator()(const char *LHS, const char *RHS) { - return strcmp(LHS, RHS) < 0; - } - }; -} - -// Note that this list cannot contain libm functions (such as acos and sqrt) -// that set errno on a domain or other error. -static const char *DoesntAccessMemoryFns[] = { - "abs", "labs", "llabs", "imaxabs", "fabs", "fabsf", "fabsl", - "trunc", "truncf", "truncl", "ldexp", - - "atan", "atanf", "atanl", "atan2", "atan2f", "atan2l", - "cbrt", - "cos", "cosf", "cosl", - "exp", "expf", "expl", - "hypot", - "sin", "sinf", "sinl", - "tan", "tanf", "tanl", "tanh", "tanhf", "tanhl", - - "floor", "floorf", "floorl", "ceil", "ceilf", "ceill", - - // ctype.h - "isalnum", "isalpha", "iscntrl", "isdigit", "isgraph", "islower", "isprint" - "ispunct", "isspace", "isupper", "isxdigit", "tolower", "toupper", - - // wctype.h" - "iswalnum", "iswalpha", "iswcntrl", "iswdigit", "iswgraph", "iswlower", - "iswprint", "iswpunct", "iswspace", "iswupper", "iswxdigit", - - "iswctype", "towctrans", "towlower", "towupper", - - "btowc", "wctob", - - "isinf", "isnan", "finite", - - // C99 math functions - "copysign", "copysignf", "copysignd", - "nexttoward", "nexttowardf", "nexttowardd", - "nextafter", "nextafterf", "nextafterd", - - // ISO C99: - "__signbit", "__signbitf", "__signbitl", -}; - - -static const char *OnlyReadsMemoryFns[] = { - "atoi", "atol", "atof", "atoll", "atoq", "a64l", - "bcmp", "memcmp", "memchr", "memrchr", "wmemcmp", "wmemchr", - - // Strings - "strcmp", "strcasecmp", "strcoll", "strncmp", "strncasecmp", - "strchr", "strcspn", "strlen", "strpbrk", "strrchr", "strspn", "strstr", - "index", "rindex", - - // Wide char strings - "wcschr", "wcscmp", "wcscoll", "wcscspn", "wcslen", "wcsncmp", "wcspbrk", - "wcsrchr", "wcsspn", "wcsstr", - - // glibc - "alphasort", "alphasort64", "versionsort", "versionsort64", - - // C99 - "nan", "nanf", "nand", - - // File I/O - "feof", "ferror", "fileno", - "feof_unlocked", "ferror_unlocked", "fileno_unlocked" -}; - -static ManagedStatic > NoMemoryTable; -static ManagedStatic > OnlyReadsMemoryTable; - -static ManagedStatic NoMemoryIntrinsics; -static ManagedStatic OnlyReadsMemoryIntrinsics; - - -AliasAnalysis::ModRefBehavior -BasicAliasAnalysis::getModRefBehavior(Function *F, CallSite CS, - std::vector *Info) { - if (!F->isDeclaration()) return UnknownModRefBehavior; - - static bool Initialized = false; - if (!Initialized) { - NoMemoryTable->insert(NoMemoryTable->end(), - DoesntAccessMemoryFns, - array_endof(DoesntAccessMemoryFns)); - - OnlyReadsMemoryTable->insert(OnlyReadsMemoryTable->end(), - OnlyReadsMemoryFns, - array_endof(OnlyReadsMemoryFns)); - - // Sort the table the first time through. - std::sort(NoMemoryTable->begin(), NoMemoryTable->end(), StringCompare()); - std::sort(OnlyReadsMemoryTable->begin(), OnlyReadsMemoryTable->end(), - StringCompare()); - - NoMemoryIntrinsics->resize(Intrinsic::num_intrinsics); - OnlyReadsMemoryIntrinsics->resize(Intrinsic::num_intrinsics); -#define GET_MODREF_BEHAVIOR -#include "llvm/Intrinsics.gen" -#undef GET_MODREF_BEHAVIOR - - Initialized = true; - } - - // If this is an intrinsic, we can use lookup tables - if (unsigned id = F->getIntrinsicID()) { - if (NoMemoryIntrinsics->test(id)) - return DoesNotAccessMemory; - if (OnlyReadsMemoryIntrinsics->test(id)) - return OnlyReadsMemory; - - return UnknownModRefBehavior; - } - - ValueName *Name = F->getValueName(); - unsigned NameLen = Name->getKeyLength(); - const char *NamePtr = Name->getKeyData(); - - // If there is an embedded nul character in the function name, we can never - // match it. - if (strlen(NamePtr) != NameLen) - return UnknownModRefBehavior; - - std::vector::iterator Ptr = - std::lower_bound(NoMemoryTable->begin(), NoMemoryTable->end(), - NamePtr, StringCompare()); - if (Ptr != NoMemoryTable->end() && strcmp(*Ptr, NamePtr) == 0) - return DoesNotAccessMemory; - - Ptr = std::lower_bound(OnlyReadsMemoryTable->begin(), - OnlyReadsMemoryTable->end(), - NamePtr, StringCompare()); - if (Ptr != OnlyReadsMemoryTable->end() && strcmp(*Ptr, NamePtr) == 0) - return OnlyReadsMemory; - - return UnknownModRefBehavior; -} - // Make sure that anything that uses AliasAnalysis pulls in this file... DEFINING_FILE_FOR(BasicAliasAnalysis)