[llvm] r367204 - [NFC][ARM][ParallelDSP] Remove PopulateLoads

[Sam Parker via llvm-commits]

Author: sam_parker
Date: Mon Jul 29 01:07:23 2019
New Revision: 367204

URL: http://llvm.org/viewvc/llvm-project?rev=367204&view=rev
Log:
[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.

Modified:
    llvm/trunk/lib/Target/ARM/ARMParallelDSP.cpp

Modified: llvm/trunk/lib/Target/ARM/ARMParallelDSP.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/ARMParallelDSP.cpp?rev=367204&r1=367203&r2=367204&view=diff
==============================================================================
--- llvm/trunk/lib/Target/ARM/ARMParallelDSP.cpp (original)
+++ llvm/trunk/lib/Target/ARM/ARMParallelDSP.cpp Mon Jul 29 01:07:23 2019
@@ -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
       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;