[llvm] r258011 - Fix buildbot failure introduced by 258010. Remove local variables became unused.
Artur Pilipenko via llvm-commits
llvm-commits at lists.llvm.org
Sun Jan 17 04:59:40 PST 2016
Author: apilipenko
Date: Sun Jan 17 06:59:40 2016
New Revision: 258011
URL: http://llvm.org/viewvc/llvm-project?rev=258011&view=rev
Log:
Fix buildbot failure introduced by 258010. Remove local variables became unused.
Modified:
llvm/trunk/lib/Transforms/Scalar/SROA.cpp
llvm/trunk/lib/Transforms/Scalar/ScalarReplAggregates.cpp
Modified: llvm/trunk/lib/Transforms/Scalar/SROA.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/SROA.cpp?rev=258011&r1=258010&r2=258011&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Scalar/SROA.cpp (original)
+++ llvm/trunk/lib/Transforms/Scalar/SROA.cpp Sun Jan 17 06:59:40 2016
@@ -1169,8 +1169,6 @@ static bool isSafePHIToSpeculate(PHINode
if (!HaveLoad)
return false;
- const DataLayout &DL = PN.getModule()->getDataLayout();
-
// We can only transform this if it is safe to push the loads into the
// predecessor blocks. The only thing to watch out for is that we can't put
// a possibly trapping load in the predecessor if it is a critical edge.
@@ -1260,7 +1258,6 @@ static void speculatePHINodeLoads(PHINod
static bool isSafeSelectToSpeculate(SelectInst &SI) {
Value *TValue = SI.getTrueValue();
Value *FValue = SI.getFalseValue();
- const DataLayout &DL = SI.getModule()->getDataLayout();
for (User *U : SI.users()) {
LoadInst *LI = dyn_cast<LoadInst>(U);
Modified: llvm/trunk/lib/Transforms/Scalar/ScalarReplAggregates.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/ScalarReplAggregates.cpp?rev=258011&r1=258010&r2=258011&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Scalar/ScalarReplAggregates.cpp (original)
+++ llvm/trunk/lib/Transforms/Scalar/ScalarReplAggregates.cpp Sun Jan 17 06:59:40 2016
@@ -1140,8 +1140,6 @@ public:
/// We can do this to a select if its only uses are loads and if the operand to
/// the select can be loaded unconditionally.
static bool isSafeSelectToSpeculate(SelectInst *SI) {
- const DataLayout &DL = SI->getModule()->getDataLayout();
-
for (User *U : SI->users()) {
LoadInst *LI = dyn_cast<LoadInst>(U);
if (!LI || !LI->isSimple()) return false;
@@ -1199,8 +1197,6 @@ static bool isSafePHIToSpeculate(PHINode
MaxAlign = std::max(MaxAlign, LI->getAlignment());
}
- const DataLayout &DL = PN->getModule()->getDataLayout();
-
// Okay, we know that we have one or more loads in the same block as the PHI.
// We can transform this if it is safe to push the loads into the predecessor
// blocks. The only thing to watch out for is that we can't put a possibly
More information about the llvm-commits
mailing list