annotate mlir/lib/Transforms/PipelineDataTransfer.cpp @ 173:0572611fdcc8 llvm10 llvm12

reorgnization done
author Shinji KONO <kono@ie.u-ryukyu.ac.jp>
date Mon, 25 May 2020 11:55:54 +0900
parents 1d019706d866
children 2e18cbf3894f
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
150
anatofuz
parents:
diff changeset
1 //===- PipelineDataTransfer.cpp --- Pass for pipelining data movement ---*-===//
anatofuz
parents:
diff changeset
2 //
anatofuz
parents:
diff changeset
3 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
anatofuz
parents:
diff changeset
4 // See https://llvm.org/LICENSE.txt for license information.
anatofuz
parents:
diff changeset
5 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
anatofuz
parents:
diff changeset
6 //
anatofuz
parents:
diff changeset
7 //===----------------------------------------------------------------------===//
anatofuz
parents:
diff changeset
8 //
anatofuz
parents:
diff changeset
9 // This file implements a pass to pipeline data transfers.
anatofuz
parents:
diff changeset
10 //
anatofuz
parents:
diff changeset
11 //===----------------------------------------------------------------------===//
anatofuz
parents:
diff changeset
12
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
13 #include "PassDetail.h"
150
anatofuz
parents:
diff changeset
14 #include "mlir/Transforms/Passes.h"
anatofuz
parents:
diff changeset
15
anatofuz
parents:
diff changeset
16 #include "mlir/Analysis/AffineAnalysis.h"
anatofuz
parents:
diff changeset
17 #include "mlir/Analysis/LoopAnalysis.h"
anatofuz
parents:
diff changeset
18 #include "mlir/Analysis/Utils.h"
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
19 #include "mlir/Dialect/Affine/IR/AffineOps.h"
150
anatofuz
parents:
diff changeset
20 #include "mlir/IR/Builders.h"
anatofuz
parents:
diff changeset
21 #include "mlir/Transforms/LoopUtils.h"
anatofuz
parents:
diff changeset
22 #include "mlir/Transforms/Utils.h"
anatofuz
parents:
diff changeset
23 #include "llvm/ADT/DenseMap.h"
anatofuz
parents:
diff changeset
24 #include "llvm/Support/Debug.h"
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
25
150
anatofuz
parents:
diff changeset
26 #define DEBUG_TYPE "affine-pipeline-data-transfer"
anatofuz
parents:
diff changeset
27
anatofuz
parents:
diff changeset
28 using namespace mlir;
anatofuz
parents:
diff changeset
29
anatofuz
parents:
diff changeset
30 namespace {
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
31 struct PipelineDataTransfer
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
32 : public AffinePipelineDataTransferBase<PipelineDataTransfer> {
150
anatofuz
parents:
diff changeset
33 void runOnFunction() override;
anatofuz
parents:
diff changeset
34 void runOnAffineForOp(AffineForOp forOp);
anatofuz
parents:
diff changeset
35
anatofuz
parents:
diff changeset
36 std::vector<AffineForOp> forOps;
anatofuz
parents:
diff changeset
37 };
anatofuz
parents:
diff changeset
38
anatofuz
parents:
diff changeset
39 } // end anonymous namespace
anatofuz
parents:
diff changeset
40
anatofuz
parents:
diff changeset
41 /// Creates a pass to pipeline explicit movement of data across levels of the
anatofuz
parents:
diff changeset
42 /// memory hierarchy.
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
43 std::unique_ptr<OperationPass<FuncOp>> mlir::createPipelineDataTransferPass() {
150
anatofuz
parents:
diff changeset
44 return std::make_unique<PipelineDataTransfer>();
anatofuz
parents:
diff changeset
45 }
anatofuz
parents:
diff changeset
46
anatofuz
parents:
diff changeset
47 // Returns the position of the tag memref operand given a DMA operation.
anatofuz
parents:
diff changeset
48 // Temporary utility: will be replaced when DmaStart/DmaFinish abstract op's are
anatofuz
parents:
diff changeset
49 // added. TODO(b/117228571)
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
50 static unsigned getTagMemRefPos(Operation &dmaOp) {
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
51 assert(isa<AffineDmaStartOp>(dmaOp) || isa<AffineDmaWaitOp>(dmaOp));
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
52 if (auto dmaStartOp = dyn_cast<AffineDmaStartOp>(dmaOp)) {
150
anatofuz
parents:
diff changeset
53 return dmaStartOp.getTagMemRefOperandIndex();
anatofuz
parents:
diff changeset
54 }
anatofuz
parents:
diff changeset
55 // First operand for a dma finish operation.
anatofuz
parents:
diff changeset
56 return 0;
anatofuz
parents:
diff changeset
57 }
anatofuz
parents:
diff changeset
58
anatofuz
parents:
diff changeset
59 /// Doubles the buffer of the supplied memref on the specified 'affine.for'
anatofuz
parents:
diff changeset
60 /// operation by adding a leading dimension of size two to the memref.
anatofuz
parents:
diff changeset
61 /// Replaces all uses of the old memref by the new one while indexing the newly
anatofuz
parents:
diff changeset
62 /// added dimension by the loop IV of the specified 'affine.for' operation
anatofuz
parents:
diff changeset
63 /// modulo 2. Returns false if such a replacement cannot be performed.
anatofuz
parents:
diff changeset
64 static bool doubleBuffer(Value oldMemRef, AffineForOp forOp) {
anatofuz
parents:
diff changeset
65 auto *forBody = forOp.getBody();
anatofuz
parents:
diff changeset
66 OpBuilder bInner(forBody, forBody->begin());
anatofuz
parents:
diff changeset
67
anatofuz
parents:
diff changeset
68 // Doubles the shape with a leading dimension extent of 2.
anatofuz
parents:
diff changeset
69 auto doubleShape = [&](MemRefType oldMemRefType) -> MemRefType {
anatofuz
parents:
diff changeset
70 // Add the leading dimension in the shape for the double buffer.
anatofuz
parents:
diff changeset
71 ArrayRef<int64_t> oldShape = oldMemRefType.getShape();
anatofuz
parents:
diff changeset
72 SmallVector<int64_t, 4> newShape(1 + oldMemRefType.getRank());
anatofuz
parents:
diff changeset
73 newShape[0] = 2;
anatofuz
parents:
diff changeset
74 std::copy(oldShape.begin(), oldShape.end(), newShape.begin() + 1);
anatofuz
parents:
diff changeset
75 return MemRefType::Builder(oldMemRefType)
anatofuz
parents:
diff changeset
76 .setShape(newShape)
anatofuz
parents:
diff changeset
77 .setAffineMaps({});
anatofuz
parents:
diff changeset
78 };
anatofuz
parents:
diff changeset
79
anatofuz
parents:
diff changeset
80 auto oldMemRefType = oldMemRef.getType().cast<MemRefType>();
anatofuz
parents:
diff changeset
81 auto newMemRefType = doubleShape(oldMemRefType);
anatofuz
parents:
diff changeset
82
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
83 // The double buffer is allocated right before 'forOp'.
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
84 OpBuilder bOuter(forOp);
150
anatofuz
parents:
diff changeset
85 // Put together alloc operands for any dynamic dimensions of the memref.
anatofuz
parents:
diff changeset
86 SmallVector<Value, 4> allocOperands;
anatofuz
parents:
diff changeset
87 unsigned dynamicDimCount = 0;
anatofuz
parents:
diff changeset
88 for (auto dimSize : oldMemRefType.getShape()) {
anatofuz
parents:
diff changeset
89 if (dimSize == -1)
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
90 allocOperands.push_back(
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
91 bOuter.create<DimOp>(forOp.getLoc(), oldMemRef, dynamicDimCount++));
150
anatofuz
parents:
diff changeset
92 }
anatofuz
parents:
diff changeset
93
anatofuz
parents:
diff changeset
94 // Create and place the alloc right before the 'affine.for' operation.
anatofuz
parents:
diff changeset
95 Value newMemRef =
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
96 bOuter.create<AllocOp>(forOp.getLoc(), newMemRefType, allocOperands);
150
anatofuz
parents:
diff changeset
97
anatofuz
parents:
diff changeset
98 // Create 'iv mod 2' value to index the leading dimension.
anatofuz
parents:
diff changeset
99 auto d0 = bInner.getAffineDimExpr(0);
anatofuz
parents:
diff changeset
100 int64_t step = forOp.getStep();
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
101 auto modTwoMap =
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
102 AffineMap::get(/*dimCount=*/1, /*symbolCount=*/0, d0.floorDiv(step) % 2);
150
anatofuz
parents:
diff changeset
103 auto ivModTwoOp = bInner.create<AffineApplyOp>(forOp.getLoc(), modTwoMap,
anatofuz
parents:
diff changeset
104 forOp.getInductionVar());
anatofuz
parents:
diff changeset
105
anatofuz
parents:
diff changeset
106 // replaceAllMemRefUsesWith will succeed unless the forOp body has
anatofuz
parents:
diff changeset
107 // non-dereferencing uses of the memref (dealloc's are fine though).
anatofuz
parents:
diff changeset
108 if (failed(replaceAllMemRefUsesWith(
anatofuz
parents:
diff changeset
109 oldMemRef, newMemRef,
anatofuz
parents:
diff changeset
110 /*extraIndices=*/{ivModTwoOp},
anatofuz
parents:
diff changeset
111 /*indexRemap=*/AffineMap(),
anatofuz
parents:
diff changeset
112 /*extraOperands=*/{},
anatofuz
parents:
diff changeset
113 /*symbolOperands=*/{},
anatofuz
parents:
diff changeset
114 /*domInstFilter=*/&*forOp.getBody()->begin()))) {
anatofuz
parents:
diff changeset
115 LLVM_DEBUG(
anatofuz
parents:
diff changeset
116 forOp.emitError("memref replacement for double buffering failed"));
anatofuz
parents:
diff changeset
117 ivModTwoOp.erase();
anatofuz
parents:
diff changeset
118 return false;
anatofuz
parents:
diff changeset
119 }
anatofuz
parents:
diff changeset
120 // Insert the dealloc op right after the for loop.
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
121 bOuter.setInsertionPointAfter(forOp);
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
122 bOuter.create<DeallocOp>(forOp.getLoc(), newMemRef);
150
anatofuz
parents:
diff changeset
123
anatofuz
parents:
diff changeset
124 return true;
anatofuz
parents:
diff changeset
125 }
anatofuz
parents:
diff changeset
126
anatofuz
parents:
diff changeset
127 /// Returns success if the IR is in a valid state.
anatofuz
parents:
diff changeset
128 void PipelineDataTransfer::runOnFunction() {
anatofuz
parents:
diff changeset
129 // Do a post order walk so that inner loop DMAs are processed first. This is
anatofuz
parents:
diff changeset
130 // necessary since 'affine.for' operations nested within would otherwise
anatofuz
parents:
diff changeset
131 // become invalid (erased) when the outer loop is pipelined (the pipelined one
anatofuz
parents:
diff changeset
132 // gets deleted and replaced by a prologue, a new steady-state loop and an
anatofuz
parents:
diff changeset
133 // epilogue).
anatofuz
parents:
diff changeset
134 forOps.clear();
anatofuz
parents:
diff changeset
135 getFunction().walk([&](AffineForOp forOp) { forOps.push_back(forOp); });
anatofuz
parents:
diff changeset
136 for (auto forOp : forOps)
anatofuz
parents:
diff changeset
137 runOnAffineForOp(forOp);
anatofuz
parents:
diff changeset
138 }
anatofuz
parents:
diff changeset
139
anatofuz
parents:
diff changeset
140 // Check if tags of the dma start op and dma wait op match.
anatofuz
parents:
diff changeset
141 static bool checkTagMatch(AffineDmaStartOp startOp, AffineDmaWaitOp waitOp) {
anatofuz
parents:
diff changeset
142 if (startOp.getTagMemRef() != waitOp.getTagMemRef())
anatofuz
parents:
diff changeset
143 return false;
anatofuz
parents:
diff changeset
144 auto startIndices = startOp.getTagIndices();
anatofuz
parents:
diff changeset
145 auto waitIndices = waitOp.getTagIndices();
anatofuz
parents:
diff changeset
146 // Both of these have the same number of indices since they correspond to the
anatofuz
parents:
diff changeset
147 // same tag memref.
anatofuz
parents:
diff changeset
148 for (auto it = startIndices.begin(), wIt = waitIndices.begin(),
anatofuz
parents:
diff changeset
149 e = startIndices.end();
anatofuz
parents:
diff changeset
150 it != e; ++it, ++wIt) {
anatofuz
parents:
diff changeset
151 // Keep it simple for now, just checking if indices match.
anatofuz
parents:
diff changeset
152 // TODO(mlir-team): this would in general need to check if there is no
anatofuz
parents:
diff changeset
153 // intervening write writing to the same tag location, i.e., memory last
anatofuz
parents:
diff changeset
154 // write/data flow analysis. This is however sufficient/powerful enough for
anatofuz
parents:
diff changeset
155 // now since the DMA generation pass or the input for it will always have
anatofuz
parents:
diff changeset
156 // start/wait with matching tags (same SSA operand indices).
anatofuz
parents:
diff changeset
157 if (*it != *wIt)
anatofuz
parents:
diff changeset
158 return false;
anatofuz
parents:
diff changeset
159 }
anatofuz
parents:
diff changeset
160 return true;
anatofuz
parents:
diff changeset
161 }
anatofuz
parents:
diff changeset
162
anatofuz
parents:
diff changeset
163 // Identify matching DMA start/finish operations to overlap computation with.
anatofuz
parents:
diff changeset
164 static void findMatchingStartFinishInsts(
anatofuz
parents:
diff changeset
165 AffineForOp forOp,
anatofuz
parents:
diff changeset
166 SmallVectorImpl<std::pair<Operation *, Operation *>> &startWaitPairs) {
anatofuz
parents:
diff changeset
167
anatofuz
parents:
diff changeset
168 // Collect outgoing DMA operations - needed to check for dependences below.
anatofuz
parents:
diff changeset
169 SmallVector<AffineDmaStartOp, 4> outgoingDmaOps;
anatofuz
parents:
diff changeset
170 for (auto &op : *forOp.getBody()) {
anatofuz
parents:
diff changeset
171 auto dmaStartOp = dyn_cast<AffineDmaStartOp>(op);
anatofuz
parents:
diff changeset
172 if (dmaStartOp && dmaStartOp.isSrcMemorySpaceFaster())
anatofuz
parents:
diff changeset
173 outgoingDmaOps.push_back(dmaStartOp);
anatofuz
parents:
diff changeset
174 }
anatofuz
parents:
diff changeset
175
anatofuz
parents:
diff changeset
176 SmallVector<Operation *, 4> dmaStartInsts, dmaFinishInsts;
anatofuz
parents:
diff changeset
177 for (auto &op : *forOp.getBody()) {
anatofuz
parents:
diff changeset
178 // Collect DMA finish operations.
anatofuz
parents:
diff changeset
179 if (isa<AffineDmaWaitOp>(op)) {
anatofuz
parents:
diff changeset
180 dmaFinishInsts.push_back(&op);
anatofuz
parents:
diff changeset
181 continue;
anatofuz
parents:
diff changeset
182 }
anatofuz
parents:
diff changeset
183 auto dmaStartOp = dyn_cast<AffineDmaStartOp>(op);
anatofuz
parents:
diff changeset
184 if (!dmaStartOp)
anatofuz
parents:
diff changeset
185 continue;
anatofuz
parents:
diff changeset
186
anatofuz
parents:
diff changeset
187 // Only DMAs incoming into higher memory spaces are pipelined for now.
anatofuz
parents:
diff changeset
188 // TODO(bondhugula): handle outgoing DMA pipelining.
anatofuz
parents:
diff changeset
189 if (!dmaStartOp.isDestMemorySpaceFaster())
anatofuz
parents:
diff changeset
190 continue;
anatofuz
parents:
diff changeset
191
anatofuz
parents:
diff changeset
192 // Check for dependence with outgoing DMAs. Doing this conservatively.
anatofuz
parents:
diff changeset
193 // TODO(andydavis,bondhugula): use the dependence analysis to check for
anatofuz
parents:
diff changeset
194 // dependences between an incoming and outgoing DMA in the same iteration.
anatofuz
parents:
diff changeset
195 auto it = outgoingDmaOps.begin();
anatofuz
parents:
diff changeset
196 for (; it != outgoingDmaOps.end(); ++it) {
anatofuz
parents:
diff changeset
197 if (it->getDstMemRef() == dmaStartOp.getSrcMemRef())
anatofuz
parents:
diff changeset
198 break;
anatofuz
parents:
diff changeset
199 }
anatofuz
parents:
diff changeset
200 if (it != outgoingDmaOps.end())
anatofuz
parents:
diff changeset
201 continue;
anatofuz
parents:
diff changeset
202
anatofuz
parents:
diff changeset
203 // We only double buffer if the buffer is not live out of loop.
anatofuz
parents:
diff changeset
204 auto memref = dmaStartOp.getOperand(dmaStartOp.getFasterMemPos());
anatofuz
parents:
diff changeset
205 bool escapingUses = false;
anatofuz
parents:
diff changeset
206 for (auto *user : memref.getUsers()) {
anatofuz
parents:
diff changeset
207 // We can double buffer regardless of dealloc's outside the loop.
anatofuz
parents:
diff changeset
208 if (isa<DeallocOp>(user))
anatofuz
parents:
diff changeset
209 continue;
anatofuz
parents:
diff changeset
210 if (!forOp.getBody()->findAncestorOpInBlock(*user)) {
anatofuz
parents:
diff changeset
211 LLVM_DEBUG(llvm::dbgs()
anatofuz
parents:
diff changeset
212 << "can't pipeline: buffer is live out of loop\n";);
anatofuz
parents:
diff changeset
213 escapingUses = true;
anatofuz
parents:
diff changeset
214 break;
anatofuz
parents:
diff changeset
215 }
anatofuz
parents:
diff changeset
216 }
anatofuz
parents:
diff changeset
217 if (!escapingUses)
anatofuz
parents:
diff changeset
218 dmaStartInsts.push_back(&op);
anatofuz
parents:
diff changeset
219 }
anatofuz
parents:
diff changeset
220
anatofuz
parents:
diff changeset
221 // For each start operation, we look for a matching finish operation.
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
222 for (auto *dmaStartOp : dmaStartInsts) {
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
223 for (auto *dmaFinishOp : dmaFinishInsts) {
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
224 if (checkTagMatch(cast<AffineDmaStartOp>(dmaStartOp),
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
225 cast<AffineDmaWaitOp>(dmaFinishOp))) {
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
226 startWaitPairs.push_back({dmaStartOp, dmaFinishOp});
150
anatofuz
parents:
diff changeset
227 break;
anatofuz
parents:
diff changeset
228 }
anatofuz
parents:
diff changeset
229 }
anatofuz
parents:
diff changeset
230 }
anatofuz
parents:
diff changeset
231 }
anatofuz
parents:
diff changeset
232
anatofuz
parents:
diff changeset
233 /// Overlap DMA transfers with computation in this loop. If successful,
anatofuz
parents:
diff changeset
234 /// 'forOp' is deleted, and a prologue, a new pipelined loop, and epilogue are
anatofuz
parents:
diff changeset
235 /// inserted right before where it was.
anatofuz
parents:
diff changeset
236 void PipelineDataTransfer::runOnAffineForOp(AffineForOp forOp) {
anatofuz
parents:
diff changeset
237 auto mayBeConstTripCount = getConstantTripCount(forOp);
anatofuz
parents:
diff changeset
238 if (!mayBeConstTripCount.hasValue()) {
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
239 LLVM_DEBUG(forOp.emitRemark("won't pipeline due to unknown trip count"));
150
anatofuz
parents:
diff changeset
240 return;
anatofuz
parents:
diff changeset
241 }
anatofuz
parents:
diff changeset
242
anatofuz
parents:
diff changeset
243 SmallVector<std::pair<Operation *, Operation *>, 4> startWaitPairs;
anatofuz
parents:
diff changeset
244 findMatchingStartFinishInsts(forOp, startWaitPairs);
anatofuz
parents:
diff changeset
245
anatofuz
parents:
diff changeset
246 if (startWaitPairs.empty()) {
anatofuz
parents:
diff changeset
247 LLVM_DEBUG(forOp.emitRemark("No dma start/finish pairs\n"));
anatofuz
parents:
diff changeset
248 return;
anatofuz
parents:
diff changeset
249 }
anatofuz
parents:
diff changeset
250
anatofuz
parents:
diff changeset
251 // Double the buffers for the higher memory space memref's.
anatofuz
parents:
diff changeset
252 // Identify memref's to replace by scanning through all DMA start
anatofuz
parents:
diff changeset
253 // operations. A DMA start operation has two memref's - the one from the
anatofuz
parents:
diff changeset
254 // higher level of memory hierarchy is the one to double buffer.
anatofuz
parents:
diff changeset
255 // TODO(bondhugula): check whether double-buffering is even necessary.
anatofuz
parents:
diff changeset
256 // TODO(bondhugula): make this work with different layouts: assuming here that
anatofuz
parents:
diff changeset
257 // the dimension we are adding here for the double buffering is the outermost
anatofuz
parents:
diff changeset
258 // dimension.
anatofuz
parents:
diff changeset
259 for (auto &pair : startWaitPairs) {
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
260 auto *dmaStartOp = pair.first;
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
261 Value oldMemRef = dmaStartOp->getOperand(
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
262 cast<AffineDmaStartOp>(dmaStartOp).getFasterMemPos());
150
anatofuz
parents:
diff changeset
263 if (!doubleBuffer(oldMemRef, forOp)) {
anatofuz
parents:
diff changeset
264 // Normally, double buffering should not fail because we already checked
anatofuz
parents:
diff changeset
265 // that there are no uses outside.
anatofuz
parents:
diff changeset
266 LLVM_DEBUG(llvm::dbgs()
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
267 << "double buffering failed for" << dmaStartOp << "\n";);
150
anatofuz
parents:
diff changeset
268 // IR still valid and semantically correct.
anatofuz
parents:
diff changeset
269 return;
anatofuz
parents:
diff changeset
270 }
anatofuz
parents:
diff changeset
271 // If the old memref has no more uses, remove its 'dead' alloc if it was
anatofuz
parents:
diff changeset
272 // alloc'ed. (note: DMA buffers are rarely function live-in; but a 'dim'
anatofuz
parents:
diff changeset
273 // operation could have been used on it if it was dynamically shaped in
anatofuz
parents:
diff changeset
274 // order to create the double buffer above.)
anatofuz
parents:
diff changeset
275 // '-canonicalize' does this in a more general way, but we'll anyway do the
anatofuz
parents:
diff changeset
276 // simple/common case so that the output / test cases looks clear.
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
277 if (auto *allocOp = oldMemRef.getDefiningOp()) {
150
anatofuz
parents:
diff changeset
278 if (oldMemRef.use_empty()) {
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
279 allocOp->erase();
150
anatofuz
parents:
diff changeset
280 } else if (oldMemRef.hasOneUse()) {
anatofuz
parents:
diff changeset
281 if (auto dealloc = dyn_cast<DeallocOp>(*oldMemRef.user_begin())) {
anatofuz
parents:
diff changeset
282 dealloc.erase();
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
283 allocOp->erase();
150
anatofuz
parents:
diff changeset
284 }
anatofuz
parents:
diff changeset
285 }
anatofuz
parents:
diff changeset
286 }
anatofuz
parents:
diff changeset
287 }
anatofuz
parents:
diff changeset
288
anatofuz
parents:
diff changeset
289 // Double the buffers for tag memrefs.
anatofuz
parents:
diff changeset
290 for (auto &pair : startWaitPairs) {
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
291 auto *dmaFinishOp = pair.second;
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
292 Value oldTagMemRef = dmaFinishOp->getOperand(getTagMemRefPos(*dmaFinishOp));
150
anatofuz
parents:
diff changeset
293 if (!doubleBuffer(oldTagMemRef, forOp)) {
anatofuz
parents:
diff changeset
294 LLVM_DEBUG(llvm::dbgs() << "tag double buffering failed\n";);
anatofuz
parents:
diff changeset
295 return;
anatofuz
parents:
diff changeset
296 }
anatofuz
parents:
diff changeset
297 // If the old tag has no uses or a single dealloc use, remove it.
anatofuz
parents:
diff changeset
298 // (canonicalization handles more complex cases).
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
299 if (auto *tagAllocOp = oldTagMemRef.getDefiningOp()) {
150
anatofuz
parents:
diff changeset
300 if (oldTagMemRef.use_empty()) {
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
301 tagAllocOp->erase();
150
anatofuz
parents:
diff changeset
302 } else if (oldTagMemRef.hasOneUse()) {
anatofuz
parents:
diff changeset
303 if (auto dealloc = dyn_cast<DeallocOp>(*oldTagMemRef.user_begin())) {
anatofuz
parents:
diff changeset
304 dealloc.erase();
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
305 tagAllocOp->erase();
150
anatofuz
parents:
diff changeset
306 }
anatofuz
parents:
diff changeset
307 }
anatofuz
parents:
diff changeset
308 }
anatofuz
parents:
diff changeset
309 }
anatofuz
parents:
diff changeset
310
anatofuz
parents:
diff changeset
311 // Double buffering would have invalidated all the old DMA start/wait insts.
anatofuz
parents:
diff changeset
312 startWaitPairs.clear();
anatofuz
parents:
diff changeset
313 findMatchingStartFinishInsts(forOp, startWaitPairs);
anatofuz
parents:
diff changeset
314
anatofuz
parents:
diff changeset
315 // Store shift for operation for later lookup for AffineApplyOp's.
anatofuz
parents:
diff changeset
316 DenseMap<Operation *, unsigned> instShiftMap;
anatofuz
parents:
diff changeset
317 for (auto &pair : startWaitPairs) {
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
318 auto *dmaStartOp = pair.first;
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
319 assert(isa<AffineDmaStartOp>(dmaStartOp));
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
320 instShiftMap[dmaStartOp] = 0;
150
anatofuz
parents:
diff changeset
321 // Set shifts for DMA start op's affine operand computation slices to 0.
anatofuz
parents:
diff changeset
322 SmallVector<AffineApplyOp, 4> sliceOps;
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
323 mlir::createAffineComputationSlice(dmaStartOp, &sliceOps);
150
anatofuz
parents:
diff changeset
324 if (!sliceOps.empty()) {
anatofuz
parents:
diff changeset
325 for (auto sliceOp : sliceOps) {
anatofuz
parents:
diff changeset
326 instShiftMap[sliceOp.getOperation()] = 0;
anatofuz
parents:
diff changeset
327 }
anatofuz
parents:
diff changeset
328 } else {
anatofuz
parents:
diff changeset
329 // If a slice wasn't created, the reachable affine.apply op's from its
anatofuz
parents:
diff changeset
330 // operands are the ones that go with it.
anatofuz
parents:
diff changeset
331 SmallVector<Operation *, 4> affineApplyInsts;
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
332 SmallVector<Value, 4> operands(dmaStartOp->getOperands());
150
anatofuz
parents:
diff changeset
333 getReachableAffineApplyOps(operands, affineApplyInsts);
anatofuz
parents:
diff changeset
334 for (auto *op : affineApplyInsts) {
anatofuz
parents:
diff changeset
335 instShiftMap[op] = 0;
anatofuz
parents:
diff changeset
336 }
anatofuz
parents:
diff changeset
337 }
anatofuz
parents:
diff changeset
338 }
anatofuz
parents:
diff changeset
339 // Everything else (including compute ops and dma finish) are shifted by one.
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
340 for (auto &op : forOp.getBody()->without_terminator())
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
341 if (instShiftMap.find(&op) == instShiftMap.end())
150
anatofuz
parents:
diff changeset
342 instShiftMap[&op] = 1;
anatofuz
parents:
diff changeset
343
anatofuz
parents:
diff changeset
344 // Get shifts stored in map.
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
345 SmallVector<uint64_t, 8> shifts(forOp.getBody()->getOperations().size());
150
anatofuz
parents:
diff changeset
346 unsigned s = 0;
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
347 for (auto &op : forOp.getBody()->without_terminator()) {
150
anatofuz
parents:
diff changeset
348 assert(instShiftMap.find(&op) != instShiftMap.end());
anatofuz
parents:
diff changeset
349 shifts[s++] = instShiftMap[&op];
anatofuz
parents:
diff changeset
350
anatofuz
parents:
diff changeset
351 // Tagging operations with shifts for debugging purposes.
anatofuz
parents:
diff changeset
352 LLVM_DEBUG({
anatofuz
parents:
diff changeset
353 OpBuilder b(&op);
anatofuz
parents:
diff changeset
354 op.setAttr("shift", b.getI64IntegerAttr(shifts[s - 1]));
anatofuz
parents:
diff changeset
355 });
anatofuz
parents:
diff changeset
356 }
anatofuz
parents:
diff changeset
357
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
358 if (!isOpwiseShiftValid(forOp, shifts)) {
150
anatofuz
parents:
diff changeset
359 // Violates dependences.
anatofuz
parents:
diff changeset
360 LLVM_DEBUG(llvm::dbgs() << "Shifts invalid - unexpected\n";);
anatofuz
parents:
diff changeset
361 return;
anatofuz
parents:
diff changeset
362 }
anatofuz
parents:
diff changeset
363
173
0572611fdcc8 reorgnization done
Shinji KONO <kono@ie.u-ryukyu.ac.jp>
parents: 150
diff changeset
364 if (failed(affineForOpBodySkew(forOp, shifts))) {
150
anatofuz
parents:
diff changeset
365 LLVM_DEBUG(llvm::dbgs() << "op body skewing failed - unexpected\n";);
anatofuz
parents:
diff changeset
366 return;
anatofuz
parents:
diff changeset
367 }
anatofuz
parents:
diff changeset
368 }