Index: llvm/trunk/include/llvm/IR/Value.h =================================================================== --- llvm/trunk/include/llvm/IR/Value.h +++ llvm/trunk/include/llvm/IR/Value.h @@ -511,6 +511,12 @@ /// dereferenceable up to the returned number of bytes. unsigned getPointerDereferenceableBytes(bool &CanBeNull) const; + /// \brief Returns true if the pointer value is fully dereferenceable. + /// + /// Sets CanBeNull to true if the pointer is either null or can be fully + /// dereferenceable. + bool isPointerDereferenceable(bool &CanBeNull) const; + /// \brief Returns an alignment of the pointer value. /// /// Returns an alignment which is either specified explicitly, e.g. via Index: llvm/trunk/lib/Analysis/Loads.cpp =================================================================== --- llvm/trunk/lib/Analysis/Loads.cpp +++ llvm/trunk/lib/Analysis/Loads.cpp @@ -91,9 +91,12 @@ // Note that it is not safe to speculate into a malloc'd region because // malloc may return null. - // These are obviously ok if aligned. - if (isa(V)) + bool CheckForNonNull; + if (V->isPointerDereferenceable(CheckForNonNull)) { + if (CheckForNonNull && !isKnownNonNullAt(V, CtxI, DT, TLI)) + return false; return isAligned(V, Align, DL); + } // It's not always safe to follow a bitcast, for example: // bitcast i8* (alloca i8) to i32* @@ -112,16 +115,6 @@ CtxI, DT, TLI, Visited); } - // Global variables which can't collapse to null are ok. - if (const GlobalVariable *GV = dyn_cast(V)) - if (!GV->hasExternalWeakLinkage()) - return isAligned(V, Align, DL); - - // byval arguments are okay. - if (const Argument *A = dyn_cast(V)) - if (A->hasByValAttr()) - return isAligned(V, Align, DL); - if (isDereferenceableFromAttribute(V, DL, CtxI, DT, TLI)) return isAligned(V, Align, DL); Index: llvm/trunk/lib/IR/Value.cpp =================================================================== --- llvm/trunk/lib/IR/Value.cpp +++ llvm/trunk/lib/IR/Value.cpp @@ -559,6 +559,29 @@ return DerefBytes; } +bool Value::isPointerDereferenceable(bool &CanBeNull) const { + assert(getType()->isPointerTy() && "must be pointer"); + + CanBeNull = false; + + // These are obviously ok. + if (isa(this)) + return true; + + // Global variables which can't collapse to null are ok. + // TODO: return true for those but set CanBeNull flag + if (const GlobalVariable *GV = dyn_cast(this)) + if (!GV->hasExternalWeakLinkage()) + return true; + + // byval arguments are okay. + if (const Argument *A = dyn_cast(this)) + if (A->hasByValAttr()) + return true; + + return false; +} + unsigned Value::getPointerAlignment(const DataLayout &DL) const { assert(getType()->isPointerTy() && "must be pointer");