Index: llvm/trunk/lib/Analysis/MemDerefPrinter.cpp =================================================================== --- llvm/trunk/lib/Analysis/MemDerefPrinter.cpp +++ llvm/trunk/lib/Analysis/MemDerefPrinter.cpp @@ -11,6 +11,7 @@ #include "llvm/ADT/SetVector.h" #include "llvm/Analysis/MemoryDependenceAnalysis.h" #include "llvm/IR/CallSite.h" +#include "llvm/IR/DataLayout.h" #include "llvm/IR/InstIterator.h" #include "llvm/IR/LLVMContext.h" #include "llvm/Support/ErrorHandling.h" @@ -25,6 +26,10 @@ MemDerefPrinter() : FunctionPass(ID) { initializeMemDerefPrinterPass(*PassRegistry::getPassRegistry()); } + void getAnalysisUsage(AnalysisUsage &AU) const override { + AU.addRequired(); + AU.setPreservesAll(); + } bool runOnFunction(Function &F) override; void print(raw_ostream &OS, const Module * = nullptr) const override; void releaseMemory() override { @@ -34,18 +39,22 @@ } char MemDerefPrinter::ID = 0; -INITIALIZE_PASS(MemDerefPrinter, "print-memderefs", - "Memory Dereferenciblity of pointers in function", false, true) +INITIALIZE_PASS_BEGIN(MemDerefPrinter, "print-memderefs", + "Memory Dereferenciblity of pointers in function", false, true) +INITIALIZE_PASS_DEPENDENCY(DataLayoutPass) +INITIALIZE_PASS_END(MemDerefPrinter, "print-memderefs", + "Memory Dereferenciblity of pointers in function", false, true) FunctionPass *llvm::createMemDerefPrinter() { return new MemDerefPrinter(); } bool MemDerefPrinter::runOnFunction(Function &F) { + const DataLayout *DL = &getAnalysis().getDataLayout(); for (auto &I: inst_range(F)) { if (LoadInst *LI = dyn_cast(&I)) { Value *PO = LI->getPointerOperand(); - if (PO->isDereferenceablePointer(nullptr)) + if (PO->isDereferenceablePointer(DL)) Vec.push_back(PO); } } Index: llvm/trunk/test/Analysis/ValueTracking/memory-dereferenceable.ll =================================================================== --- llvm/trunk/test/Analysis/ValueTracking/memory-dereferenceable.ll +++ llvm/trunk/test/Analysis/ValueTracking/memory-dereferenceable.ll @@ -3,11 +3,13 @@ ; Uses the print-deref (+ analyze to print) pass to run ; isDereferenceablePointer() on many load instruction operands +target datalayout = "e" + declare zeroext i1 @return_i1() @globalstr = global [6 x i8] c"hello\00" -define void @test(i32 addrspace(1)* byval %dparam) { +define void @test(i32 addrspace(1)* dereferenceable(8) %dparam) { ; CHECK: The following are dereferenceable: ; CHECK: %globalptr ; CHECK: %alloca