[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