diff options
author | Daniel Berlin <dberlin@dberlin.org> | 2017-03-11 00:51:01 +0000 |
---|---|---|
committer | Daniel Berlin <dberlin@dberlin.org> | 2017-03-11 00:51:01 +0000 |
commit | cd07a0f685149018d48adfc660b4ab8ee0c43e81 (patch) | |
tree | 87d3902eb5279fabe71c50369eeaede11ef0e0a9 | |
parent | 79da2a7698ab2e86d6eef12859d879ccfb98f46e (diff) | |
download | bcm5719-llvm-cd07a0f685149018d48adfc660b4ab8ee0c43e81.tar.gz bcm5719-llvm-cd07a0f685149018d48adfc660b4ab8ee0c43e81.zip |
VNCoercion: Make the function signatures all consistent
llvm-svn: 297537
-rw-r--r-- | llvm/include/llvm/Transforms/Utils/VNCoercion.h | 2 | ||||
-rw-r--r-- | llvm/lib/Transforms/Scalar/GVN.cpp | 2 | ||||
-rw-r--r-- | llvm/lib/Transforms/Utils/VNCoercion.cpp | 3 |
3 files changed, 3 insertions, 4 deletions
diff --git a/llvm/include/llvm/Transforms/Utils/VNCoercion.h b/llvm/include/llvm/Transforms/Utils/VNCoercion.h index d3c998fa8a8..edc63ca38db 100644 --- a/llvm/include/llvm/Transforms/Utils/VNCoercion.h +++ b/llvm/include/llvm/Transforms/Utils/VNCoercion.h @@ -53,7 +53,7 @@ Value *coerceAvailableValueToLoadType(Value *StoredVal, Type *LoadedTy, /// On success, it returns the offset into DepSI that extraction would start. /// On failure, it returns -1. int analyzeLoadFromClobberingStore(Type *LoadTy, Value *LoadPtr, - StoreInst *DepSI); + StoreInst *DepSI, const DataLayout &DL); /// This function determines whether a value for the pointer LoadPtr can be /// extracted from the load at DepLI. diff --git a/llvm/lib/Transforms/Scalar/GVN.cpp b/llvm/lib/Transforms/Scalar/GVN.cpp index b5953106167..1208311fa3c 100644 --- a/llvm/lib/Transforms/Scalar/GVN.cpp +++ b/llvm/lib/Transforms/Scalar/GVN.cpp @@ -832,7 +832,7 @@ bool GVN::AnalyzeLoadAvailability(LoadInst *LI, MemDepResult DepInfo, // Can't forward from non-atomic to atomic without violating memory model. if (Address && LI->isAtomic() <= DepSI->isAtomic()) { int Offset = - analyzeLoadFromClobberingStore(LI->getType(), Address, DepSI); + analyzeLoadFromClobberingStore(LI->getType(), Address, DepSI, DL); if (Offset != -1) { Res = AvailableValue::get(DepSI->getValueOperand(), Offset); return true; diff --git a/llvm/lib/Transforms/Utils/VNCoercion.cpp b/llvm/lib/Transforms/Utils/VNCoercion.cpp index 38d26e922c3..b317aa7e38a 100644 --- a/llvm/lib/Transforms/Utils/VNCoercion.cpp +++ b/llvm/lib/Transforms/Utils/VNCoercion.cpp @@ -190,13 +190,12 @@ static int analyzeLoadFromClobberingWrite(Type *LoadTy, Value *LoadPtr, /// This function is called when we have a /// memdep query of a load that ends up being a clobbering store. int analyzeLoadFromClobberingStore(Type *LoadTy, Value *LoadPtr, - StoreInst *DepSI) { + StoreInst *DepSI, const DataLayout &DL) { // Cannot handle reading from store of first-class aggregate yet. if (DepSI->getValueOperand()->getType()->isStructTy() || DepSI->getValueOperand()->getType()->isArrayTy()) return -1; - const DataLayout &DL = DepSI->getModule()->getDataLayout(); Value *StorePtr = DepSI->getPointerOperand(); uint64_t StoreSize = DL.getTypeSizeInBits(DepSI->getValueOperand()->getType()); |