[llvm] 8c01deb - [ARMParallelDSP] Remove unnecessary wrapper function (NFC)
Nikita Popov via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 23 06:28:06 PDT 2021
Author: Nikita Popov
Date: 2021-06-23T15:27:54+02:00
New Revision: 8c01deb8e65490d730e8a2eac1e4ffa8822ce992
URL: https://github.com/llvm/llvm-project/commit/8c01deb8e65490d730e8a2eac1e4ffa8822ce992
DIFF: https://github.com/llvm/llvm-project/commit/8c01deb8e65490d730e8a2eac1e4ffa8822ce992.diff
LOG: [ARMParallelDSP] Remove unnecessary wrapper function (NFC)
AreSequentialAccesses() forwards directly to isConsecutiveAccess()
and has an unnecessary template parameter to boot.
Added:
Modified:
llvm/lib/Target/ARM/ARMParallelDSP.cpp
Removed:
################################################################################
diff --git a/llvm/lib/Target/ARM/ARMParallelDSP.cpp b/llvm/lib/Target/ARM/ARMParallelDSP.cpp
index 9a7c1f541aa24..46baf89309399 100644
--- a/llvm/lib/Target/ARM/ARMParallelDSP.cpp
+++ b/llvm/lib/Target/ARM/ARMParallelDSP.cpp
@@ -299,14 +299,6 @@ namespace {
};
}
-template<typename MemInst>
-static bool AreSequentialAccesses(MemInst *MemOp0, MemInst *MemOp1,
- const DataLayout &DL, ScalarEvolution &SE) {
- if (isConsecutiveAccess(MemOp0, MemOp1, DL, SE))
- return true;
- return false;
-}
-
bool ARMParallelDSP::AreSequentialLoads(LoadInst *Ld0, LoadInst *Ld1,
MemInstList &VecMem) {
if (!Ld0 || !Ld1)
@@ -414,7 +406,7 @@ bool ARMParallelDSP::RecordMemoryOps(BasicBlock *BB) {
if (Base == Offset || OffsetLoads.count(Offset))
continue;
- if (AreSequentialAccesses<LoadInst>(Base, Offset, *DL, *SE) &&
+ if (isConsecutiveAccess(Base, Offset, *DL, *SE) &&
SafeToPair(Base, Offset)) {
LoadPairs[Base] = Offset;
OffsetLoads.insert(Offset);
More information about the llvm-commits
mailing list