[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