[PATCH] D48872: [ARM][NFC] Refactor sequential access for DSP

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 3 04:59:07 PDT 2018


SjoerdMeijer accepted this revision.
SjoerdMeijer added a comment.
This revision is now accepted and ready to land.

Looks OK to me.



================
Comment at: lib/Target/ARM/ARMParallelDSP.cpp:61
     ValueList    VL;        // List of all (narrow) operands of this Mul
-    LoadInstList VecLd;     // List of all load instructions of this Mul
+    MemInstList  VecLd;     // List of all load instructions of this Mul
     MemLocList   MemLocs;   // All memory locations read by this Mul
----------------
Nit: does the comment now needs updating? I.e., that it can be any instruction touching memory?


https://reviews.llvm.org/D48872





More information about the llvm-commits mailing list