[NFC][ARM][ParallelDSP] Remove PopulateLoads

We no longer have to check what loads are used, all this
is performed at the start of the transform, so it's not
doing anything now.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@367204 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/Target/ARM/ARMParallelDSP.cpp b/lib/Target/ARM/ARMParallelDSP.cpp
index 0e48ba4c7a4a5bba3342e81d04e8b64657ce2596..7b357193f71eb7a452a86730eadaf5e98880d4f6 100644 (file)
--- a/lib/Target/ARM/ARMParallelDSP.cpp
+++ b/lib/Target/ARM/ARMParallelDSP.cpp
@@ -64,7 +64,6 @@ namespace {
   struct BinOpChain {
     Instruction   *Root;
     ValueList     AllValues;
-    MemInstList   Loads;
     MemInstList   VecLd;    // List of all load instructions.
     ValueList     LHS;      // List of all (narrow) left hand operands.
     ValueList     RHS;      // List of all (narrow) right hand operands.
@@ -79,13 +78,6 @@ namespace {
           AllValues.push_back(V);
     }
 
-    void PopulateLoads() {
-      for (auto *V : AllValues) {
-        if (auto *Ld = dyn_cast<LoadInst>(V))
-          Loads.push_back(Ld);
-      }
-    }
-
     unsigned size() const { return AllValues.size(); }
 
     bool AreSymmetrical(BinOpChain *Other);
@@ -563,7 +555,6 @@ bool ARMParallelDSP::CreateParallelPairs(Reduction &R) {
       LLVM_DEBUG(dbgs() << "Operand list too short.\n");
       return false;
     }
-    MulChain->PopulateLoads();
     ValueList &LHS = static_cast<BinOpChain*>(MulChain.get())->LHS;
     ValueList &RHS = static_cast<BinOpChain*>(MulChain.get())->RHS;