WIP

[hunhoffe]

No description provided.

[github-actions]

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

clang-format

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitDepthPromotion.cpp

Line 330 in 3e39cad

if (tileIt != inferredMap.end() && !tileIt->second.consumerTiles.empty()) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitDmaTaskToConduit.cpp

Lines 86 to 91 in 3e39cad

	static void
	dimsToOffsetsStrides(int32_t bdOffset, int64_t len,
	AIE::BDDimLayoutArrayAttr dimensions,
	llvm::SmallVectorImpl<int64_t> &offsets,
	llvm::SmallVectorImpl<int64_t> &sizes,
	llvm::SmallVectorImpl<int64_t> &strides) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitDmaTaskToConduit.cpp

Lines 146 to 148 in 3e39cad

	llvm::StringRef conduitName =
	conduitChannelAttr ? conduitChannelAttr.getValue()
	: alloc.getSymName();

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitDmaTaskToConduit.cpp

Lines 161 to 164 in 3e39cad

	void processRuntimeSequence(
	AIE::RuntimeSequenceOp rtSeq,
	const llvm::StringMap<ShimAllocInfo> &allocMap,
	mlir::OpBuilder &builder) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitDmaTaskToConduit.cpp

Line 210 in 3e39cad

dimsToOffsetsStrides(bdOffset, len, dimensions, offsets, sizes, strides);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitDmaTaskToConduit.cpp

Lines 225 to 226 in 3e39cad

	offsetsAttr, sizesAttr, stridesAttr,
	dimensions);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseChannels.cpp

Lines 365 to 366 in 3e39cad

	auto nameAttr =
	op->getAttrOfType<mlir::FlatSymbolRefAttr>("name");

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseChannels.cpp

Lines 511 to 512 in 3e39cad

	auto nameAttr =
	op->getAttrOfType<mlir::FlatSymbolRefAttr>("name");

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseCoreBodyPass.cpp

Line 24 in 3e39cad

// direct memref.alloc references (L1) or relay buffer references (MemTile).

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseCoreBodyPass.cpp

Line 127 in 3e39cad

mlir::MLIRContext *ctx) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseCoreBodyPass.cpp

Line 377 in 3e39cad

mlir::MLIRContext *ctx) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseCoreBodyPass.cpp

Lines 425 to 426 in 3e39cad

	static llvm::SmallVector<std::string>
	getProducedChannels(AIE::CoreOp core) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseCoreBodyPass.cpp

Lines 447 to 448 in 3e39cad

	static llvm::SmallVector<std::string>
	getConsumedChannels(AIE::CoreOp core) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseCoreBodyPass.cpp

Lines 563 to 565 in 3e39cad

	device.walk([&](Create createOp) {
	createMap[createOp.getName()] = createOp;
	});

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseCoreBodyPass.cpp

Lines 623 to 624 in 3e39cad

	std::string intermediateName =
	intermediateConduit.getName().str();

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseCoreBodyPass.cpp

Lines 707 to 709 in 3e39cad

	static IntermediateRoute
	decideRoute(Create intermediateConduit, mlir::Value tile,
	AIE::DeviceOp device) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseCoreBodyPass.cpp

Lines 741 to 744 in 3e39cad

	static std::string emitMemTileRelay(FusableCorePair &pair,
	AIE::DeviceOp device,
	mlir::OpBuilder &builder,
	mlir::MLIRContext *ctx) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseCoreBodyPass.cpp

Lines 756 to 768 in 3e39cad

	builder.create<Create>(
	intermediateConduit.getLoc(), mlir::StringAttr::get(ctx, relayName),
	intermediateConduit.getElementTypeAttr(),
	intermediateConduit.getDepthAttr(),
	/*routing_mode=*/RoutingModeAttr{},
	/*sync_mode=*/SyncModeAttr{},
	/*producer_rates=*/nullptr,
	/*consumer_rates=*/nullptr,
	/*fusion_group=*/mlir::StringAttr{},
	/*bd_repeat=*/nullptr,
	/*dma_repeat=*/nullptr,
	/*producer_dimensions=*/nullptr,
	/*consumer_dimensions=*/nullptr);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseCoreBodyPass.cpp

Lines 779 to 780 in 3e39cad

	mlir::ArrayAttr dstsArr = mlir::ArrayAttr::get(
	ctx, {mlir::FlatSymbolRefAttr::get(ctx, relayName)});

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseCoreBodyPass.cpp

Lines 791 to 794 in 3e39cad

	static mlir::LogicalResult composeCoresBodies(
	FusableCorePair &pair, IntermediateRoute route,
	mlir::OpBuilder &builder, mlir::MLIRContext *ctx,
	AIE::DeviceOp device) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseCoreBodyPass.cpp

Line 857 in 3e39cad

mlir::Value allocVal; // L1 route only.

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseCoreBodyPass.cpp

Lines 942 to 943 in 3e39cad

	mapping.map(consumerFor.getInductionVar(),
	producerFor.getInductionVar());

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseCoreBodyPass.cpp

Lines 960 to 961 in 3e39cad

	defOp != consumerFor.getOperation() &&
	!mapping.contains(operand) && alreadyCloned.insert(defOp).second) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseCoreBodyPass.cpp

Lines 1050 to 1051 in 3e39cad

	mlir::Value deadToken,
	mlir::OpBuilder &builder) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseCoreBodyPass.cpp

Line 1066 in 3e39cad

resultToken.getUsers().end());

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseCoreBodyPass.cpp

Line 1094 in 3e39cad

mlir::OpBuilder &builder) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseCoreBodyPass.cpp

Line 1102 in 3e39cad

token.getUsers().end());

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseCoreBodyPass.cpp

Lines 1113 to 1114 in 3e39cad

	mlir::OpBuilder &builder,
	const std::string &consumerChannelName) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseCoreBodyPass.cpp

Lines 1315 to 1316 in 3e39cad

	std::string channelName =
	pair.intermediateConduit.getName().str();

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseOperators.cpp

Line 139 in 3e39cad

bool hasShimConsumer = tileIt != inferredMap.end() && !tileIt->second.shimConsumerTiles.empty();

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseOperators.cpp

Lines 263 to 264 in 3e39cad

	static llvm::SmallVector<ArgGroup>
	buildArgGroupsFromSeq(mlir::Block &seqBody) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseOperators.cpp

Lines 276 to 277 in 3e39cad

	int64_t offset =
	(offsetsAttr && !offsetsAttr.empty()) ? offsetsAttr[0] : 0;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseOperators.cpp

Lines 621 to 622 in 3e39cad

	auto nameAttr =
	op->getAttrOfType<mlir::FlatSymbolRefAttr>("name");

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseOperators.cpp

Lines 653 to 654 in 3e39cad

	Key k = {alloc.getTile(),
	static_cast<int>(alloc.getChannelDir())};

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseOperators.cpp

Lines 664 to 665 in 3e39cad

	if (allocs[idx].getChannelIndex() !=
	static_cast<int64_t>(idx))

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseOperators.cpp

Lines 912 to 915 in 3e39cad

	auto computeDeadArgs = [](
	const llvm::SmallVector<ArgGroup> &groups,
	const llvm::StringSet<> &erasedChannels,
	unsigned numOrigArgs) -> llvm::DenseSet<unsigned> {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseOperators.cpp

Lines 934 to 939 in 3e39cad

	llvm::DenseSet<unsigned> deadA = computeDeadArgs(
	argGroupsA, erasedChannelsA,
	static_cast<unsigned>(origTypesA.size()));
	llvm::DenseSet<unsigned> deadB = computeDeadArgs(
	argGroupsB, erasedChannelsB,
	static_cast<unsigned>(origTypesB.size()));

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitInferModes.cpp

Lines 246 to 247 in 3e39cad

	op.setRoutingModeAttr(
	RoutingModeAttr::get(module.getContext(), RoutingMode::SharedMemory));

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitInferModes.cpp

Lines 308 to 309 in 3e39cad

	unsigned numConsumers = consCoords.empty() ? 1
	: consCoords.size();

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitPlaceBuffers.cpp

Line 64 in 3e39cad

if (rest.empty() || rest.find_first_not_of("0123456789") != llvm::StringRef::npos)

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitTileInference.cpp

Lines 335 to 336 in 3e39cad

	if (!createOp->getAttrOfType<mlir::StringAttr>(
	"dma_channel_group"))

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMAAlloc.cpp

Line 512 in 3e39cad

// Multi-device: allocate into the device that owns the producer tile.

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMAAlloc.cpp

Lines 666 to 667 in 3e39cad

	if (targetModel.isMemTile(consCol, consRow) &&
	info.putCount > 1 && info.dmaRepeat == 0) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMAAlloc.cpp

Lines 763 to 764 in 3e39cad

	auto consLocks =
	state.allocateLockPair(consTileVal, consPrefix, consNBufs, prodInit);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMACollect.cpp

Lines 137 to 138 in 3e39cad

	info.producerDimensions =
	mlir::cast<AIE::BDDimLayoutArrayAttr>(*dims);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMACollect.cpp

Lines 140 to 141 in 3e39cad

	auto arrayOfArrays =
	mlir::cast<AIE::BDDimLayoutArrayArrayAttr>(*dims);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMACollect.cpp

Lines 234 to 235 in 3e39cad

	for (int devIdx = 0;
	devIdx < static_cast<int>(state.deviceOps.size()); ++devIdx) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMACollect.cpp

Lines 421 to 422 in 3e39cad

	auto arrayOfArrays =
	mlir::cast<AIE::BDDimLayoutArrayArrayAttr>(*dims);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMACollect.cpp

Line 532 in 3e39cad

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMACommon.h

Lines 128 to 129 in 3e39cad

	<< count << " IDs (aligned to " << p
	<< ") but only " << (unsigned)(limit - next) << " IDs remain";

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMACommon.h

Lines 672 to 673 in 3e39cad

	llvm::DenseMap<mlir::Value,
	std::pair<mlir::Value, mlir::Value>> pktTileS2MMLock;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMACommon.h

Lines 744 to 745 in 3e39cad

	if (activeDevIdx >= 0 &&
	activeDevIdx < static_cast<int>(deviceOps.size()))

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMACommon.h

Lines 877 to 879 in 3e39cad

	uint32_t maxLocks = targetModel->getNumLocks(
	static_cast<int>(tileOp.getCol()),
	static_cast<int>(tileOp.getRow()));

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMACommon.h

Lines 881 to 883 in 3e39cad

	int currentUsed = lockIdCounter.count(tileVal)
	? lockIdCounter[tileVal]
	: 0;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMACommon.h

Lines 891 to 892 in 3e39cad

	llvm::Twine(static_cast<int>(maxLocks) - currentUsed) +
	" of " + llvm::Twine(maxLocks) + " remain");

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMACommon.h

Lines 945 to 946 in 3e39cad

	ConduitInfo *lookupConduit(mlir::StringRef name,
	mlir::Operation *contextOp) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 469 to 471 in 3e39cad

	// perBufLen: number of elements per physical buffer for this source conduit.
	// Prefer numElems (from put_memref_async descriptors), then derive from
	// elemType (e.g. memref<48xi32> → 48), otherwise fall back to 1.

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 517 to 518 in 3e39cad

	if (ConduitInfo *dstInfoR =
	state.lookupConduit(dstNameR, linkOp.op))

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 840 to 841 in 3e39cad

	state.pktTileS2MMLock[consTileVal] = {
	locks.first.getResult(), locks.second.getResult()};

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 893 to 894 in 3e39cad

	srcPort =
	state.tileNextMM2SChannel[srcProdTile.getResult()]++;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 1232 to 1233 in 3e39cad

	if (ConduitInfo *dstInfo =
	state.lookupConduit(dstName2, linkOp.op)) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 1856 to 1857 in 3e39cad

	std::string qFG = state.qualifyFuseGroup(info.fuseGroup,
	info.deviceIndex);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 1916 to 1917 in 3e39cad

	bdTermBlock =
	(info.dmaRepeat > 0) ? addBlock() : nullptr;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 2382 to 2383 in 3e39cad

	// first conduit emits a dma_start. Subsequent conduits append BD blocks
	// and the post-pass links all BD chains into a single combined ring.

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Line 2398 in 3e39cad

std::to_string(consRow) + "_ch" + std::to_string(s2mmChannel),

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 2439 to 2440 in 3e39cad

	// Linear chain condition: either dma_repeat>0 (finite DMA task queue),
	// or putCount>1 with no dmaRepeat (N sequential puts merged by

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 2442 to 2444 in 3e39cad

	isLinearChain =
	(info.dmaRepeat > 0) ||
	(info.putCount > 1 && info.dmaRepeat == 0);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 2508 to 2509 in 3e39cad

	builder.create<AIE::NextBDOp>(state.deviceOp.getLoc(),
	bdBlocks[0]);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 2529 to 2530 in 3e39cad

	std::string bdKey =
	name + "__s2mm_" + std::to_string(consIdx);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 2537 to 2540 in 3e39cad

	std::string bdKey =
	name + "__s2mm_" + std::to_string(consIdx);
	std::string qS2MM = state.qualifyFuseGroup(info.fuseGroupS2MM,
	info.deviceIndex);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 729 to 730 in 3e39cad

	auto ctrKey =
	std::make_tuple(conduitName.str(), col, row, false);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 741 to 742 in 3e39cad

	initBuilder.create<mlir::arith::ConstantIndexOp>(
	loc, rotationBufSlot);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 755 to 756 in 3e39cad

	auto ctrKey =
	std::make_tuple(conduitName.str(), col, row, true);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 760 to 761 in 3e39cad

	initBuilder.setInsertionPointAfterValue(
	resolvedProducerRotationBuf);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 770 to 772 in 3e39cad

	initBuilder.create<mlir::memref::StoreOp>(
	loc, zero, resolvedProducerRotationBuf,
	mlir::ValueRange{slotIdx});

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1044 to 1047 in 3e39cad

	// This is the reverse of --dma-task-to-conduit. After --conduit-fuse-operators
	// merges runtime_sequences and eliminates dead block args, the remaining
	// put/get ops correspond 1:1 (positionally) to the runtime_sequence block
	// args. Each Nth conduit.put_memref/get_memref maps to block arg N.

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1063 to 1064 in 3e39cad

	if (auto cc = alloc->getAttrOfType<mlir::FlatSymbolRefAttr>(
	"conduit_channel"))

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1105 to 1106 in 3e39cad

	bool isS2MM =
	(conduitToDir[conduitName] == AIE::DMAChannelDir::S2MM);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1117 to 1119 in 3e39cad

	if (auto dimsAttr =
	op->getAttrOfType<AIE::BDDimLayoutArrayAttr>(
	"producer_dimensions"))

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1126 to 1129 in 3e39cad

	mlir::OperationState configState(loc,
	"aiex.dma_configure_task_for");
	configState.addAttribute(
	"alloc", mlir::FlatSymbolRefAttr::get(ctx, allocSym));

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1131 to 1132 in 3e39cad

	configState.addAttribute("issue_token",
	builder.getBoolAttr(true));

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1144 to 1146 in 3e39cad

	builder.create<AIE::DMABDOp>(
	loc, bufArg, /*offset=*/0,
	static_cast<int>(numElems), dims);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1148 to 1150 in 3e39cad

	builder.create<AIE::DMABDOp>(
	loc, bufArg, /*offset=*/0,
	static_cast<int>(numElems));

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1152 to 1153 in 3e39cad

	bdBlock->back().setAttr("burst_length",
	builder.getI32IntegerAttr(0));

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1175 to 1176 in 3e39cad

	mlir::OperationState awaitState(rtSeq.getLoc(),
	"aiex.dma_await_task");

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1183 to 1184 in 3e39cad

	mlir::OperationState freeState(rtSeq.getLoc(),
	"aiex.dma_free_task");

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMAPass.cpp

Lines 330 to 339 in 3e39cad

	// Skip cores that already have link_with or link_files set.
	if (coreOp.getLinkWith() || coreOp.getLinkFiles())
	return;
	// Check if any func.call inside this core references a function
	// with link_with.
	std::string linkWithValue;
	coreOp.walk([&](mlir::func::CallOp callOp) {
	auto it = funcLinkWith.find(callOp.getCallee());
	if (it != funcLinkWith.end() && linkWithValue.empty())
	linkWithValue = it->second;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMAPass.cpp

Lines 341 to 344 in 3e39cad

	if (!linkWithValue.empty())
	coreOp.setLinkWithAttr(
	mlir::StringAttr::get(module.getContext(), linkWithValue));
	});

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 249 in 3e39cad

locks.second.getResult()};

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 434 to 435 in 3e39cad

	if (!isAIE2 && !info.externalBuffers.empty() &&
	!info.noLocks) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 484 to 485 in 3e39cad

	std::string qS2MM = state.qualifyFuseGroup(info.fuseGroupS2MM,
	info.deviceIndex);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 529 to 530 in 3e39cad

	llvm::Twine(consCol) + "," + llvm::Twine(consRow) +
	"): all " + llvm::Twine(maxS2MM_4a) + " channels in use");

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 726 in 3e39cad

llvm::StringMap<unsigned> fuseGroupPacketIDNext; // qFG → next member index

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 783 to 786 in 3e39cad

	for (auto &[name, info] : state.conduitMap) {
	// Multi-device: ensure tile lookups target the correct device.
	if (state.isMultiDevice())
	state.switchToDeviceIndex(info.deviceIndex);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 788 to 790 in 3e39cad

	// Pass 0: only packet conduits. Pass 1: everything else.
	if (flowPass == 0 && info.routingMode != "packet") continue;
	if (flowPass == 1 && info.routingMode == "packet") continue;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 792 to 807 in 3e39cad

	if (info.routingMode == "cascade")
	continue;
	if (info.sharedMemory)
	continue;
	if (state.linkSrcNamesEarly.count(name) ||
	state.linkJoinSrcNames.count(name))
	continue;
	// Link destinations: flows are emitted by linkPhase() — skip here to
	// avoid duplicate flows.
	if (state.linkDstNames.count(name))
	continue;
	auto [prodCol, prodRow] = info.producerTileCoord;
	if (prodCol < 0 || prodRow == 0)
	continue;
	if (info.consumerTileCoords.empty())
	continue;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 809 to 812 in 3e39cad

	AIE::TileOp prodTile = state.lookupTileByCoord(prodCol, prodRow);
	if (!prodTile)
	continue;
	mlir::Value prodTileVal = prodTile.getResult();

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 814 to 815 in 3e39cad

	if (!info.consumerTileBuffers.count(prodTileVal))
	continue;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 817 in 3e39cad

builder.setInsertionPoint(state.deviceBody->getTerminator());

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 819 to 825 in 3e39cad

	// ---- Determine hardware MM2S channel count for this tile. ----
	// Used by the mode=any exhaustion check (Step 3.5).
	uint32_t maxMM2S = 2; // hardware default: 2 MM2S per compute tile
	if (state.targetModel)
	maxMM2S = state.targetModel->getNumSourceSwitchboxConnections(
	static_cast<int>(prodCol), static_cast<int>(prodRow),
	AIE::WireBundle::DMA);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 827 to 837 in 3e39cad

	// ---- Assign MM2S channel (fused groups share a channel). ----
	// Check if Phase 4b already assigned an MM2S channel for this conduit
	// (happens when the conduit has both compute and shim consumers — the
	// shim consumer flow and the compute consumer flow share the same
	// producer-side MM2S channel as a hardware broadcast).
	int32_t mm2sChannel = -1;
	bool usedPacketFallback = false;
	{
	auto existingIt = state.conduitMM2SChannel.find(name);
	if (existingIt != state.conduitMM2SChannel.end()) {
	mm2sChannel = existingIt->second;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 839 in 3e39cad

}

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 841 to 876 in 3e39cad

	if (mm2sChannel >= 0) {
	// Already assigned by Phase 4b — reuse (broadcast from same MM2S port).
	} else if (!info.fuseGroup.empty()) {
	std::string qFG = state.qualifyFuseGroup(info.fuseGroup,
	info.deviceIndex);
	auto it = state.fuseGroupMM2SChannel.find(qFG);
	if (it != state.fuseGroupMM2SChannel.end()) {
	mm2sChannel = it->second;
	} else {
	mm2sChannel = state.tileNextMM2SChannel[prodTileVal]++;
	state.fuseGroupMM2SChannel[qFG] = mm2sChannel;
	}
	state.fuseGroupMembers[qFG].push_back(name);
	state.conduitMM2SChannel[name] = mm2sChannel;
	} else if (info.routingMode == "any") {
	// mode=any: check whether a circuit DMA channel is available.
	int32_t nextCh = state.tileNextMM2SChannel.count(prodTileVal)
	? state.tileNextMM2SChannel[prodTileVal]
	: 0;
	if (static_cast<uint32_t>(nextCh) < maxMM2S) {
	// A free circuit-mode channel exists — use it.
	mm2sChannel = state.tileNextMM2SChannel[prodTileVal]++;
	state.conduitMM2SChannel[name] = mm2sChannel;
	} else {
	// Circuit DMA exhausted; flag that Step 3.5 handles emission below.
	usedPacketFallback = true;
	}
	} else {
	// Check if a circuit DMA channel is available before allocating.
	// If exhausted, fall back to packet-switched DMA for non-cascade/
	// non-shared-memory channels (extends the mode=any fallback).
	int32_t nextCh = state.tileNextMM2SChannel.count(prodTileVal)
	? state.tileNextMM2SChannel[prodTileVal]
	: 0;
	if (static_cast<uint32_t>(nextCh) < maxMM2S) {
	mm2sChannel = state.tileNextMM2SChannel[prodTileVal]++;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 878 to 884 in 3e39cad

	// Track packet-mode channel designation for Step 3.5c.
	if (info.routingMode == "packet" && prodTile) {
	auto key = std::make_pair(prodTile.getOperation(),
	static_cast<int>(mm2sChannel));
	state.pktChannelState.isPacketChannel[key] = true;
	// NOTE: usedPacketFallback is NOT set here; explicit packet-mode
	// broadcast is handled below (single multi-dest packet flow).

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 886 to 893 in 3e39cad

	} else if (info.routingMode != "cascade" &&
	info.routingMode != "shared_memory" &&
	info.routingMode != "stream") {
	// Circuit DMA exhausted on producer tile — fall back to
	// packet-switched DMA regardless of explicit routing_mode.
	// tryPacketFallback (Step 3.5c) will reuse an existing
	// packet-designated MM2S channel if one exists.
	usedPacketFallback = true;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 895 to 897 in 3e39cad

	// Cascade/shared_memory/stream cannot use packet fallback.
	mm2sChannel = state.tileNextMM2SChannel[prodTileVal]++;
	state.conduitMM2SChannel[name] = mm2sChannel;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 899 in 3e39cad

}

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 901 to 924 in 3e39cad

	// ---- Explicit packet-mode broadcast: single multi-dest flow. ----
	// For routing_mode="packet", emit one aie.packet_flow with all consumer
	// destinations and a single packet ID. The switchbox hardware broadcasts
	// each packet to all destinations. This matches the oracle's behavior
	// (AIEObjectFifoStatefulTransform) where one bdPacket ID is used for all
	// producer MM2S BDs and one packet_flow carries multiple packet_dest ops.
	if (info.routingMode == "packet" && mm2sChannel >= 0 &&
	!info.consumerTileCoords.empty()) {
	if (!state.packetIDAllocator) {
	state.module.emitError(
	"internal error: packetIDAllocator not initialized");
	state.passFailed = true;
	return;
	}
	// Use pre-allocated aligned block ID if this channel belongs to a fuse
	// group with multiple packet members; otherwise fall back to sequential.
	std::optional<uint8_t> pktID;
	std::string qFG;
	if (!info.fuseGroup.empty()) {
	qFG = state.qualifyFuseGroup(info.fuseGroup, info.deviceIndex);
	auto baseIt = fuseGroupPacketIDBase.find(qFG);
	if (baseIt != fuseGroupPacketIDBase.end()) {
	unsigned idx = fuseGroupPacketIDNext[qFG]++;
	pktID = static_cast<uint8_t>(baseIt->second + idx);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 926 in 3e39cad

}

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 927 to 946 in 3e39cad

	if (!pktID) {
	mlir::Value pktDomain = state.getMemTileDomain(prodTileVal);
	pktID = state.packetIDAllocator->allocate(pktDomain);
	}
	if (!pktID) {
	state.passFailed = true;
	return;
	}
	state.conduitPacketID[name] = *pktID;
	auto pktFlow = builder.create<AIE::PacketFlowOp>(
	state.deviceOp.getLoc(), static_cast<int8_t>(*pktID),
	/*keep_pkt_header=*/mlir::BoolAttr{},
	/*priority_route=*/mlir::BoolAttr{});
	mlir::Region &region = pktFlow.getPorts();
	mlir::Block *pktBlock = builder.createBlock(&region);
	builder.setInsertionPointToStart(pktBlock);
	builder.create<AIE::PacketSourceOp>(state.deviceOp.getLoc(), prodTileVal,
	AIE::WireBundle::DMA,
	static_cast<int32_t>(mm2sChannel));

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 948 in 3e39cad

for (unsigned consIdx = 0; consIdx < info.consumerTileCoords.size();

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 954 in 3e39cad

AIE::TileOp consTile = state.lookupTileByCoord(consCol, consRow);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 957 in 3e39cad

mlir::Value consTileVal = consTile.getResult();

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 959 to 988 in 3e39cad

	// Allocate S2MM channel on the consumer tile.
	// Only share S2MM ports between packet channels that have the same
	// dma_channel_group. Independent packet channels (no group) get
	// separate S2MM ports to prevent data crossover (e.g., Q/K vs V in
	// flash attention — same tile, different data).
	// Effective group key: dma_channel_group_s2mm if set, else
	// dma_channel_group.
	std::string s2mmGrp = state.qualifyFuseGroup(
	!info.fuseGroupS2MM.empty() ? info.fuseGroupS2MM
	: info.fuseGroup,
	info.deviceIndex);
	int32_t s2mmChannel;
	bool s2mmShared = false;
	if (!s2mmGrp.empty()) {
	auto it = state.fuseGroupS2MMChannel.find(s2mmGrp);
	if (it != state.fuseGroupS2MMChannel.end()) {
	s2mmChannel = it->second;
	s2mmShared = true;
	}
	}
	if (!s2mmShared) {
	uint32_t maxS2MM_pkt = 2;
	if (state.targetModel)
	maxS2MM_pkt = state.targetModel->getNumDestSwitchboxConnections(
	static_cast<int>(consCol), static_cast<int>(consRow),
	AIE::WireBundle::DMA);
	int32_t nextS2MM_pkt = state.tileNextS2MMChannel.count(consTileVal)
	? state.tileNextS2MMChannel[consTileVal]
	: 0;
	if (static_cast<uint32_t>(nextS2MM_pkt) >= maxS2MM_pkt) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 990 to 993 in 3e39cad

	llvm::Twine("conduit-to-dma: S2MM DMA channel exhausted on "
	"tile (") +
	llvm::Twine(consCol) + "," + llvm::Twine(consRow) +
	"): all " + llvm::Twine(maxS2MM_pkt) + " channels in use");

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 997 to 1059 in 3e39cad

	s2mmChannel = state.tileNextS2MMChannel[consTileVal]++;
	if (!s2mmGrp.empty())
	state.fuseGroupS2MMChannel[s2mmGrp] = s2mmChannel;
	}
	state.conduitConsS2MMChannel[{name, consIdx}] = s2mmChannel;
	// Lock sharing: only share locks when channels share an S2MM port
	// (same dma_channel_group). Independent channels use separate locks.
	if (s2mmShared) {
	auto lockIt = state.pktTileS2MMLock.find(consTileVal);
	if (lockIt != state.pktTileS2MMLock.end()) {
	info.consumerTileLocks[consTileVal] = {
	lockIt->second.first.getDefiningOp<AIE::LockOp>(),
	lockIt->second.second.getDefiningOp<AIE::LockOp>()};
	}
	} else if (!s2mmGrp.empty()) {
	// First in group: record locks for future group members.
	auto &locks = info.consumerTileLocks[consTileVal];
	if (locks.first && locks.second) {
	state.pktTileS2MMLock[consTileVal] = {locks.first.getResult(),
	locks.second.getResult()};
	}
	}
	builder.create<AIE::PacketDestOp>(state.deviceOp.getLoc(), consTileVal,
	AIE::WireBundle::DMA,
	static_cast<int32_t>(s2mmChannel));
	// Record the emitted port pair so that fuse group partners sharing
	// the same MM2S+S2MM ports do not emit a duplicate circuit flow.
	FlowKey fk{prodTileVal.getAsOpaquePointer(), mm2sChannel,
	consTileVal.getAsOpaquePointer(), s2mmChannel};
	emittedFlowPorts.insert(fk);
	}
	builder.create<AIE::EndOp>(state.deviceOp.getLoc());
	builder.setInsertionPointAfter(pktFlow);
	continue; // skip per-consumer circuit/fallback flow loop
	}
	// ---- Emit flows per consumer. ----
	for (unsigned consIdx = 0; consIdx < info.consumerTileCoords.size();
	++consIdx) {
	auto [consCol, consRow] = info.consumerTileCoords[consIdx];
	if (consRow == 0)
	continue;
	// For single-consumer conduits, adjacent tiles use shared memory
	// (Phase 3c) — no DMA flow needed. For broadcast (multi-consumer),
	// Phase 3c is skipped; all consumers use DMA, so flows are needed
	// for every consumer regardless of adjacency.
	// Exception: forceDMA forces DMA even for adjacent tiles.
	// Also: when shim consumers exist, the producer needs DMA MM2S
	// regardless (to reach the shim tile via the switchbox network),
	// so the compute consumer flow must also be emitted.
	if (!info.forceDMA && info.consumerTileCoords.size() == 1 &&
	info.shimConsumerTileCoords.empty()) {
	bool explicitSharedMem = (info.routingMode == "shared_memory");
	bool rightAdj = state.targetModel->isLegalMemAffinity(prodCol, prodRow,
	consCol, consRow);
	bool leftAdj = state.targetModel->isLegalMemAffinity(consCol, consRow,
	prodCol, prodRow);
	if (explicitSharedMem || rightAdj || leftAdj)

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 1061 to 1087 in 3e39cad

	}
	AIE::TileOp consTile = state.lookupTileByCoord(consCol, consRow);
	if (!consTile)
	continue;
	mlir::Value consTileVal = consTile.getResult();
	if (usedPacketFallback) {
	// Step 3.5: attempt packet DMA fallback.
	bool ok =
	tryPacketFallback(state, name, info, prodTileVal, prodCol, prodRow,
	consTileVal, consCol, consRow, consIdx);
	if (!ok) {
	// Step 4: all modes exhausted — emit a hard error.
	state.deviceOp.emitError(
	llvm::Twine("conduit-to-dma: no DMA resources available for "
	"conduit '") +
	name + "': circuit DMA MM2S channels exhausted on tile (" +
	llvm::Twine(prodCol) + "," + llvm::Twine(prodRow) +
	") and packet DMA fallback is also ineligible "
	"(check BD budget, lock budget, and packet flow ID budget)");
	// B-3 fix: return immediately so the outer conduit loop does not
	// continue processing subsequent conduits with broken state after
	// both circuit DMA and packet fallback have been exhausted.
	state.passFailed = true;
	return;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 1089 to 1092 in 3e39cad

	// tryPacketFallback records conduitMM2SChannel and
	// conduitConsS2MMChannel internally; skip the circuit path below.
	continue;
	}

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 1094 to 1102 in 3e39cad

	// S2MM fuse group: reuse existing S2MM channel if another conduit
	// in the same fuse group already allocated one on this tile.
	int32_t s2mmChannel;
	if (!info.fuseGroupS2MM.empty()) {
	std::string qS2MM = state.qualifyFuseGroup(info.fuseGroupS2MM,
	info.deviceIndex);
	auto it = state.fuseGroupS2MMChannel.find(qS2MM);
	if (it != state.fuseGroupS2MMChannel.end()) {
	s2mmChannel = it->second;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 1104 in 3e39cad

uint32_t maxS2MM_4c = 2;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 1116 to 1117 in 3e39cad

	llvm::Twine(consCol) + "," + llvm::Twine(consRow) +
	"): all " + llvm::Twine(maxS2MM_4c) + " channels in use");

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 1122 to 1143 in 3e39cad

	state.fuseGroupS2MMChannel[qS2MM] = s2mmChannel;
	}
	std::string bdKey = name + "__s2mm_" + std::to_string(consIdx);
	state.fuseGroupMembers[qS2MM].push_back(bdKey);
	} else {
	// Bounds-check S2MM channels on the consumer tile.
	uint32_t maxS2MM_4c = 2;
	if (state.targetModel)
	maxS2MM_4c = state.targetModel->getNumDestSwitchboxConnections(
	static_cast<int>(consCol), static_cast<int>(consRow),
	AIE::WireBundle::DMA);
	int32_t nextS2MM_4c = state.tileNextS2MMChannel.count(consTileVal)
	? state.tileNextS2MMChannel[consTileVal]
	: 0;
	if (static_cast<uint32_t>(nextS2MM_4c) >= maxS2MM_4c) {
	state.deviceOp.emitError(
	llvm::Twine("conduit-to-dma: S2MM DMA channel exhausted on "
	"tile (") +
	llvm::Twine(consCol) + "," + llvm::Twine(consRow) +
	"): all " + llvm::Twine(maxS2MM_4c) + " channels in use");
	state.passFailed = true;
	return;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 1145 to 1157 in 3e39cad

	s2mmChannel = state.tileNextS2MMChannel[consTileVal]++;
	}
	state.conduitConsS2MMChannel[{name, consIdx}] = s2mmChannel;
	// Deduplicate: skip if the same source→dest port pair was already
	// emitted by the packet broadcast path or a fuse group partner.
	// This prevents duplicate packet_flow + aie.flow for the same ports
	// when MM2S fuse group members share the same consumer S2MM channel.
	FlowKey fk{prodTileVal.getAsOpaquePointer(), mm2sChannel,
	consTileVal.getAsOpaquePointer(), s2mmChannel};
	if (emittedFlowPorts.count(fk))
	continue;
	emittedFlowPorts.insert(fk);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 1159 to 1161 in 3e39cad

	state.emitFlow(info.routingMode, prodTileVal, AIE::WireBundle::DMA,
	mm2sChannel, consTileVal, AIE::WireBundle::DMA,
	s2mmChannel);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 1163 in 3e39cad

}

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ObjectFifoToConduit.cpp

Lines 1053 to 1054 in 3e39cad

	loc, elemTy,
	mlir::FlatSymbolRefAttr::get(ctx, name));

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ObjectFifoToConduit.cpp

Lines 1271 to 1272 in 3e39cad

	loc, storedVal,
	mlir::FlatSymbolRefAttr::get(ctx, name));

[github-actions]

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

clang-format

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseOperators.cpp

Lines 263 to 264 in 39c4268

	static llvm::SmallVector<ArgGroup>
	buildArgGroupsFromSeq(mlir::Block &seqBody) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseOperators.cpp

Lines 276 to 277 in 39c4268

	int64_t offset =
	(offsetsAttr && !offsetsAttr.empty()) ? offsetsAttr[0] : 0;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseOperators.cpp

Lines 621 to 622 in 39c4268

	auto nameAttr =
	op->getAttrOfType<mlir::FlatSymbolRefAttr>("name");

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseOperators.cpp

Lines 653 to 654 in 39c4268

	Key k = {alloc.getTile(),
	static_cast<int>(alloc.getChannelDir())};

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseOperators.cpp

Lines 664 to 665 in 39c4268

	if (allocs[idx].getChannelIndex() !=
	static_cast<int64_t>(idx))

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseOperators.cpp

Lines 912 to 915 in 39c4268

	auto computeDeadArgs = [](
	const llvm::SmallVector<ArgGroup> &groups,
	const llvm::StringSet<> &erasedChannels,
	unsigned numOrigArgs) -> llvm::DenseSet<unsigned> {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitFuseOperators.cpp

Lines 934 to 939 in 39c4268

	llvm::DenseSet<unsigned> deadA = computeDeadArgs(
	argGroupsA, erasedChannelsA,
	static_cast<unsigned>(origTypesA.size()));
	llvm::DenseSet<unsigned> deadB = computeDeadArgs(
	argGroupsB, erasedChannelsB,
	static_cast<unsigned>(origTypesB.size()));

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitInferModes.cpp

Lines 246 to 247 in 39c4268

	op.setRoutingModeAttr(
	RoutingModeAttr::get(module.getContext(), RoutingMode::SharedMemory));

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitInferModes.cpp

Lines 308 to 309 in 39c4268

	unsigned numConsumers = consCoords.empty() ? 1
	: consCoords.size();

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitPlaceBuffers.cpp

Line 64 in 39c4268

if (rest.empty() || rest.find_first_not_of("0123456789") != llvm::StringRef::npos)

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitTileInference.cpp

Lines 335 to 336 in 39c4268

	if (!createOp->getAttrOfType<mlir::StringAttr>(
	"dma_channel_group"))

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMAAlloc.cpp

Line 512 in 39c4268

// Multi-device: allocate into the device that owns the producer tile.

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMAAlloc.cpp

Lines 666 to 667 in 39c4268

	if (targetModel.isMemTile(consCol, consRow) &&
	info.putCount > 1 && info.dmaRepeat == 0) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMAAlloc.cpp

Lines 763 to 764 in 39c4268

	auto consLocks =
	state.allocateLockPair(consTileVal, consPrefix, consNBufs, prodInit);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMACollect.cpp

Lines 137 to 138 in 39c4268

	info.producerDimensions =
	mlir::cast<AIE::BDDimLayoutArrayAttr>(*dims);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMACollect.cpp

Lines 140 to 141 in 39c4268

	auto arrayOfArrays =
	mlir::cast<AIE::BDDimLayoutArrayArrayAttr>(*dims);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMACollect.cpp

Lines 234 to 235 in 39c4268

	for (int devIdx = 0;
	devIdx < static_cast<int>(state.deviceOps.size()); ++devIdx) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMACollect.cpp

Lines 421 to 422 in 39c4268

	auto arrayOfArrays =
	mlir::cast<AIE::BDDimLayoutArrayArrayAttr>(*dims);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMACollect.cpp

Line 532 in 39c4268

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMACommon.h

Lines 128 to 129 in 39c4268

	<< count << " IDs (aligned to " << p
	<< ") but only " << (unsigned)(limit - next) << " IDs remain";

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMACommon.h

Lines 672 to 673 in 39c4268

	llvm::DenseMap<mlir::Value,
	std::pair<mlir::Value, mlir::Value>> pktTileS2MMLock;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMACommon.h

Lines 744 to 745 in 39c4268

	if (activeDevIdx >= 0 &&
	activeDevIdx < static_cast<int>(deviceOps.size()))

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMACommon.h

Lines 877 to 879 in 39c4268

	uint32_t maxLocks = targetModel->getNumLocks(
	static_cast<int>(tileOp.getCol()),
	static_cast<int>(tileOp.getRow()));

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMACommon.h

Lines 881 to 883 in 39c4268

	int currentUsed = lockIdCounter.count(tileVal)
	? lockIdCounter[tileVal]
	: 0;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMACommon.h

Lines 891 to 892 in 39c4268

	llvm::Twine(static_cast<int>(maxLocks) - currentUsed) +
	" of " + llvm::Twine(maxLocks) + " remain");

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMACommon.h

Lines 945 to 946 in 39c4268

	ConduitInfo *lookupConduit(mlir::StringRef name,
	mlir::Operation *contextOp) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 469 to 471 in 39c4268

	// perBufLen: number of elements per physical buffer for this source conduit.
	// Prefer numElems (from put_memref_async descriptors), then derive from
	// elemType (e.g. memref<48xi32> → 48), otherwise fall back to 1.

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 517 to 518 in 39c4268

	if (ConduitInfo *dstInfoR =
	state.lookupConduit(dstNameR, linkOp.op))

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 840 to 841 in 39c4268

	state.pktTileS2MMLock[consTileVal] = {
	locks.first.getResult(), locks.second.getResult()};

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 893 to 894 in 39c4268

	srcPort =
	state.tileNextMM2SChannel[srcProdTile.getResult()]++;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 1232 to 1233 in 39c4268

	if (ConduitInfo *dstInfo =
	state.lookupConduit(dstName2, linkOp.op)) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 1856 to 1857 in 39c4268

	std::string qFG = state.qualifyFuseGroup(info.fuseGroup,
	info.deviceIndex);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 1916 to 1917 in 39c4268

	bdTermBlock =
	(info.dmaRepeat > 0) ? addBlock() : nullptr;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 2382 to 2383 in 39c4268

	// first conduit emits a dma_start. Subsequent conduits append BD blocks
	// and the post-pass links all BD chains into a single combined ring.

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Line 2398 in 39c4268

std::to_string(consRow) + "_ch" + std::to_string(s2mmChannel),

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 2439 to 2440 in 39c4268

	// Linear chain condition: either dma_repeat>0 (finite DMA task queue),
	// or putCount>1 with no dmaRepeat (N sequential puts merged by

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 2442 to 2444 in 39c4268

	isLinearChain =
	(info.dmaRepeat > 0) ||
	(info.putCount > 1 && info.dmaRepeat == 0);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 2508 to 2509 in 39c4268

	builder.create<AIE::NextBDOp>(state.deviceOp.getLoc(),
	bdBlocks[0]);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 2529 to 2530 in 39c4268

	std::string bdKey =
	name + "__s2mm_" + std::to_string(consIdx);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 2537 to 2540 in 39c4268

	std::string bdKey =
	name + "__s2mm_" + std::to_string(consIdx);
	std::string qS2MM = state.qualifyFuseGroup(info.fuseGroupS2MM,
	info.deviceIndex);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 729 to 730 in 39c4268

	auto ctrKey =
	std::make_tuple(conduitName.str(), col, row, false);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 741 to 742 in 39c4268

	initBuilder.create<mlir::arith::ConstantIndexOp>(
	loc, rotationBufSlot);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 755 to 756 in 39c4268

	auto ctrKey =
	std::make_tuple(conduitName.str(), col, row, true);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 760 to 761 in 39c4268

	initBuilder.setInsertionPointAfterValue(
	resolvedProducerRotationBuf);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 770 to 772 in 39c4268

	initBuilder.create<mlir::memref::StoreOp>(
	loc, zero, resolvedProducerRotationBuf,
	mlir::ValueRange{slotIdx});

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1044 to 1047 in 39c4268

	// This is the reverse of --dma-task-to-conduit. After --conduit-fuse-operators
	// merges runtime_sequences and eliminates dead block args, the remaining
	// put/get ops correspond 1:1 (positionally) to the runtime_sequence block
	// args. Each Nth conduit.put_memref/get_memref maps to block arg N.

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1063 to 1064 in 39c4268

	if (auto cc = alloc->getAttrOfType<mlir::FlatSymbolRefAttr>(
	"conduit_channel"))

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1105 to 1106 in 39c4268

	bool isS2MM =
	(conduitToDir[conduitName] == AIE::DMAChannelDir::S2MM);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1117 to 1119 in 39c4268

	if (auto dimsAttr =
	op->getAttrOfType<AIE::BDDimLayoutArrayAttr>(
	"producer_dimensions"))

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1126 to 1129 in 39c4268

	mlir::OperationState configState(loc,
	"aiex.dma_configure_task_for");
	configState.addAttribute(
	"alloc", mlir::FlatSymbolRefAttr::get(ctx, allocSym));

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1131 to 1132 in 39c4268

	configState.addAttribute("issue_token",
	builder.getBoolAttr(true));

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1144 to 1146 in 39c4268

	builder.create<AIE::DMABDOp>(
	loc, bufArg, /*offset=*/0,
	static_cast<int>(numElems), dims);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1148 to 1150 in 39c4268

	builder.create<AIE::DMABDOp>(
	loc, bufArg, /*offset=*/0,
	static_cast<int>(numElems));

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1152 to 1153 in 39c4268

	bdBlock->back().setAttr("burst_length",
	builder.getI32IntegerAttr(0));

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1175 to 1176 in 39c4268

	mlir::OperationState awaitState(rtSeq.getLoc(),
	"aiex.dma_await_task");

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1183 to 1184 in 39c4268

	mlir::OperationState freeState(rtSeq.getLoc(),
	"aiex.dma_free_task");

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMAPass.cpp

Lines 330 to 339 in 39c4268

	// Skip cores that already have link_with or link_files set.
	if (coreOp.getLinkWith() || coreOp.getLinkFiles())
	return;
	// Check if any func.call inside this core references a function
	// with link_with.
	std::string linkWithValue;
	coreOp.walk([&](mlir::func::CallOp callOp) {
	auto it = funcLinkWith.find(callOp.getCallee());
	if (it != funcLinkWith.end() && linkWithValue.empty())
	linkWithValue = it->second;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMAPass.cpp

Lines 341 to 344 in 39c4268

	if (!linkWithValue.empty())
	coreOp.setLinkWithAttr(
	mlir::StringAttr::get(module.getContext(), linkWithValue));
	});

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 249 in 39c4268

locks.second.getResult()};

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 434 to 435 in 39c4268

	if (!isAIE2 && !info.externalBuffers.empty() &&
	!info.noLocks) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 484 to 485 in 39c4268

	std::string qS2MM = state.qualifyFuseGroup(info.fuseGroupS2MM,
	info.deviceIndex);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 529 to 530 in 39c4268

	llvm::Twine(consCol) + "," + llvm::Twine(consRow) +
	"): all " + llvm::Twine(maxS2MM_4a) + " channels in use");

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 726 in 39c4268

llvm::StringMap<unsigned> fuseGroupPacketIDNext; // qFG → next member index

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 783 to 786 in 39c4268

	for (auto &[name, info] : state.conduitMap) {
	// Multi-device: ensure tile lookups target the correct device.
	if (state.isMultiDevice())
	state.switchToDeviceIndex(info.deviceIndex);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 788 to 790 in 39c4268

	// Pass 0: only packet conduits. Pass 1: everything else.
	if (flowPass == 0 && info.routingMode != "packet") continue;
	if (flowPass == 1 && info.routingMode == "packet") continue;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 792 to 807 in 39c4268

	if (info.routingMode == "cascade")
	continue;
	if (info.sharedMemory)
	continue;
	if (state.linkSrcNamesEarly.count(name) ||
	state.linkJoinSrcNames.count(name))
	continue;
	// Link destinations: flows are emitted by linkPhase() — skip here to
	// avoid duplicate flows.
	if (state.linkDstNames.count(name))
	continue;
	auto [prodCol, prodRow] = info.producerTileCoord;
	if (prodCol < 0 || prodRow == 0)
	continue;
	if (info.consumerTileCoords.empty())
	continue;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 809 to 812 in 39c4268

	AIE::TileOp prodTile = state.lookupTileByCoord(prodCol, prodRow);
	if (!prodTile)
	continue;
	mlir::Value prodTileVal = prodTile.getResult();

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 814 to 815 in 39c4268

	if (!info.consumerTileBuffers.count(prodTileVal))
	continue;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 817 in 39c4268

builder.setInsertionPoint(state.deviceBody->getTerminator());

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 819 to 825 in 39c4268

	// ---- Determine hardware MM2S channel count for this tile. ----
	// Used by the mode=any exhaustion check (Step 3.5).
	uint32_t maxMM2S = 2; // hardware default: 2 MM2S per compute tile
	if (state.targetModel)
	maxMM2S = state.targetModel->getNumSourceSwitchboxConnections(
	static_cast<int>(prodCol), static_cast<int>(prodRow),
	AIE::WireBundle::DMA);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 827 to 837 in 39c4268

	// ---- Assign MM2S channel (fused groups share a channel). ----
	// Check if Phase 4b already assigned an MM2S channel for this conduit
	// (happens when the conduit has both compute and shim consumers — the
	// shim consumer flow and the compute consumer flow share the same
	// producer-side MM2S channel as a hardware broadcast).
	int32_t mm2sChannel = -1;
	bool usedPacketFallback = false;
	{
	auto existingIt = state.conduitMM2SChannel.find(name);
	if (existingIt != state.conduitMM2SChannel.end()) {
	mm2sChannel = existingIt->second;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 839 in 39c4268

}

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 841 to 876 in 39c4268

	if (mm2sChannel >= 0) {
	// Already assigned by Phase 4b — reuse (broadcast from same MM2S port).
	} else if (!info.fuseGroup.empty()) {
	std::string qFG = state.qualifyFuseGroup(info.fuseGroup,
	info.deviceIndex);
	auto it = state.fuseGroupMM2SChannel.find(qFG);
	if (it != state.fuseGroupMM2SChannel.end()) {
	mm2sChannel = it->second;
	} else {
	mm2sChannel = state.tileNextMM2SChannel[prodTileVal]++;
	state.fuseGroupMM2SChannel[qFG] = mm2sChannel;
	}
	state.fuseGroupMembers[qFG].push_back(name);
	state.conduitMM2SChannel[name] = mm2sChannel;
	} else if (info.routingMode == "any") {
	// mode=any: check whether a circuit DMA channel is available.
	int32_t nextCh = state.tileNextMM2SChannel.count(prodTileVal)
	? state.tileNextMM2SChannel[prodTileVal]
	: 0;
	if (static_cast<uint32_t>(nextCh) < maxMM2S) {
	// A free circuit-mode channel exists — use it.
	mm2sChannel = state.tileNextMM2SChannel[prodTileVal]++;
	state.conduitMM2SChannel[name] = mm2sChannel;
	} else {
	// Circuit DMA exhausted; flag that Step 3.5 handles emission below.
	usedPacketFallback = true;
	}
	} else {
	// Check if a circuit DMA channel is available before allocating.
	// If exhausted, fall back to packet-switched DMA for non-cascade/
	// non-shared-memory channels (extends the mode=any fallback).
	int32_t nextCh = state.tileNextMM2SChannel.count(prodTileVal)
	? state.tileNextMM2SChannel[prodTileVal]
	: 0;
	if (static_cast<uint32_t>(nextCh) < maxMM2S) {
	mm2sChannel = state.tileNextMM2SChannel[prodTileVal]++;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 878 to 884 in 39c4268

	// Track packet-mode channel designation for Step 3.5c.
	if (info.routingMode == "packet" && prodTile) {
	auto key = std::make_pair(prodTile.getOperation(),
	static_cast<int>(mm2sChannel));
	state.pktChannelState.isPacketChannel[key] = true;
	// NOTE: usedPacketFallback is NOT set here; explicit packet-mode
	// broadcast is handled below (single multi-dest packet flow).

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 886 to 893 in 39c4268

	} else if (info.routingMode != "cascade" &&
	info.routingMode != "shared_memory" &&
	info.routingMode != "stream") {
	// Circuit DMA exhausted on producer tile — fall back to
	// packet-switched DMA regardless of explicit routing_mode.
	// tryPacketFallback (Step 3.5c) will reuse an existing
	// packet-designated MM2S channel if one exists.
	usedPacketFallback = true;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 895 to 897 in 39c4268

	// Cascade/shared_memory/stream cannot use packet fallback.
	mm2sChannel = state.tileNextMM2SChannel[prodTileVal]++;
	state.conduitMM2SChannel[name] = mm2sChannel;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 899 in 39c4268

}

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 901 to 924 in 39c4268

	// ---- Explicit packet-mode broadcast: single multi-dest flow. ----
	// For routing_mode="packet", emit one aie.packet_flow with all consumer
	// destinations and a single packet ID. The switchbox hardware broadcasts
	// each packet to all destinations. This matches the oracle's behavior
	// (AIEObjectFifoStatefulTransform) where one bdPacket ID is used for all
	// producer MM2S BDs and one packet_flow carries multiple packet_dest ops.
	if (info.routingMode == "packet" && mm2sChannel >= 0 &&
	!info.consumerTileCoords.empty()) {
	if (!state.packetIDAllocator) {
	state.module.emitError(
	"internal error: packetIDAllocator not initialized");
	state.passFailed = true;
	return;
	}
	// Use pre-allocated aligned block ID if this channel belongs to a fuse
	// group with multiple packet members; otherwise fall back to sequential.
	std::optional<uint8_t> pktID;
	std::string qFG;
	if (!info.fuseGroup.empty()) {
	qFG = state.qualifyFuseGroup(info.fuseGroup, info.deviceIndex);
	auto baseIt = fuseGroupPacketIDBase.find(qFG);
	if (baseIt != fuseGroupPacketIDBase.end()) {
	unsigned idx = fuseGroupPacketIDNext[qFG]++;
	pktID = static_cast<uint8_t>(baseIt->second + idx);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 926 in 39c4268

}

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 927 to 946 in 39c4268

	if (!pktID) {
	mlir::Value pktDomain = state.getMemTileDomain(prodTileVal);
	pktID = state.packetIDAllocator->allocate(pktDomain);
	}
	if (!pktID) {
	state.passFailed = true;
	return;
	}
	state.conduitPacketID[name] = *pktID;
	auto pktFlow = builder.create<AIE::PacketFlowOp>(
	state.deviceOp.getLoc(), static_cast<int8_t>(*pktID),
	/*keep_pkt_header=*/mlir::BoolAttr{},
	/*priority_route=*/mlir::BoolAttr{});
	mlir::Region &region = pktFlow.getPorts();
	mlir::Block *pktBlock = builder.createBlock(&region);
	builder.setInsertionPointToStart(pktBlock);
	builder.create<AIE::PacketSourceOp>(state.deviceOp.getLoc(), prodTileVal,
	AIE::WireBundle::DMA,
	static_cast<int32_t>(mm2sChannel));

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 948 in 39c4268

for (unsigned consIdx = 0; consIdx < info.consumerTileCoords.size();

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 954 in 39c4268

AIE::TileOp consTile = state.lookupTileByCoord(consCol, consRow);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 957 in 39c4268

mlir::Value consTileVal = consTile.getResult();

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 959 to 988 in 39c4268

	// Allocate S2MM channel on the consumer tile.
	// Only share S2MM ports between packet channels that have the same
	// dma_channel_group. Independent packet channels (no group) get
	// separate S2MM ports to prevent data crossover (e.g., Q/K vs V in
	// flash attention — same tile, different data).
	// Effective group key: dma_channel_group_s2mm if set, else
	// dma_channel_group.
	std::string s2mmGrp = state.qualifyFuseGroup(
	!info.fuseGroupS2MM.empty() ? info.fuseGroupS2MM
	: info.fuseGroup,
	info.deviceIndex);
	int32_t s2mmChannel;
	bool s2mmShared = false;
	if (!s2mmGrp.empty()) {
	auto it = state.fuseGroupS2MMChannel.find(s2mmGrp);
	if (it != state.fuseGroupS2MMChannel.end()) {
	s2mmChannel = it->second;
	s2mmShared = true;
	}
	}
	if (!s2mmShared) {
	uint32_t maxS2MM_pkt = 2;
	if (state.targetModel)
	maxS2MM_pkt = state.targetModel->getNumDestSwitchboxConnections(
	static_cast<int>(consCol), static_cast<int>(consRow),
	AIE::WireBundle::DMA);
	int32_t nextS2MM_pkt = state.tileNextS2MMChannel.count(consTileVal)
	? state.tileNextS2MMChannel[consTileVal]
	: 0;
	if (static_cast<uint32_t>(nextS2MM_pkt) >= maxS2MM_pkt) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 990 to 993 in 39c4268

	llvm::Twine("conduit-to-dma: S2MM DMA channel exhausted on "
	"tile (") +
	llvm::Twine(consCol) + "," + llvm::Twine(consRow) +
	"): all " + llvm::Twine(maxS2MM_pkt) + " channels in use");

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 997 to 1059 in 39c4268

	s2mmChannel = state.tileNextS2MMChannel[consTileVal]++;
	if (!s2mmGrp.empty())
	state.fuseGroupS2MMChannel[s2mmGrp] = s2mmChannel;
	}
	state.conduitConsS2MMChannel[{name, consIdx}] = s2mmChannel;
	// Lock sharing: only share locks when channels share an S2MM port
	// (same dma_channel_group). Independent channels use separate locks.
	if (s2mmShared) {
	auto lockIt = state.pktTileS2MMLock.find(consTileVal);
	if (lockIt != state.pktTileS2MMLock.end()) {
	info.consumerTileLocks[consTileVal] = {
	lockIt->second.first.getDefiningOp<AIE::LockOp>(),
	lockIt->second.second.getDefiningOp<AIE::LockOp>()};
	}
	} else if (!s2mmGrp.empty()) {
	// First in group: record locks for future group members.
	auto &locks = info.consumerTileLocks[consTileVal];
	if (locks.first && locks.second) {
	state.pktTileS2MMLock[consTileVal] = {locks.first.getResult(),
	locks.second.getResult()};
	}
	}
	builder.create<AIE::PacketDestOp>(state.deviceOp.getLoc(), consTileVal,
	AIE::WireBundle::DMA,
	static_cast<int32_t>(s2mmChannel));
	// Record the emitted port pair so that fuse group partners sharing
	// the same MM2S+S2MM ports do not emit a duplicate circuit flow.
	FlowKey fk{prodTileVal.getAsOpaquePointer(), mm2sChannel,
	consTileVal.getAsOpaquePointer(), s2mmChannel};
	emittedFlowPorts.insert(fk);
	}
	builder.create<AIE::EndOp>(state.deviceOp.getLoc());
	builder.setInsertionPointAfter(pktFlow);
	continue; // skip per-consumer circuit/fallback flow loop
	}
	// ---- Emit flows per consumer. ----
	for (unsigned consIdx = 0; consIdx < info.consumerTileCoords.size();
	++consIdx) {
	auto [consCol, consRow] = info.consumerTileCoords[consIdx];
	if (consRow == 0)
	continue;
	// For single-consumer conduits, adjacent tiles use shared memory
	// (Phase 3c) — no DMA flow needed. For broadcast (multi-consumer),
	// Phase 3c is skipped; all consumers use DMA, so flows are needed
	// for every consumer regardless of adjacency.
	// Exception: forceDMA forces DMA even for adjacent tiles.
	// Also: when shim consumers exist, the producer needs DMA MM2S
	// regardless (to reach the shim tile via the switchbox network),
	// so the compute consumer flow must also be emitted.
	if (!info.forceDMA && info.consumerTileCoords.size() == 1 &&
	info.shimConsumerTileCoords.empty()) {
	bool explicitSharedMem = (info.routingMode == "shared_memory");
	bool rightAdj = state.targetModel->isLegalMemAffinity(prodCol, prodRow,
	consCol, consRow);
	bool leftAdj = state.targetModel->isLegalMemAffinity(consCol, consRow,
	prodCol, prodRow);
	if (explicitSharedMem || rightAdj || leftAdj)

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 1061 to 1087 in 39c4268

	}
	AIE::TileOp consTile = state.lookupTileByCoord(consCol, consRow);
	if (!consTile)
	continue;
	mlir::Value consTileVal = consTile.getResult();
	if (usedPacketFallback) {
	// Step 3.5: attempt packet DMA fallback.
	bool ok =
	tryPacketFallback(state, name, info, prodTileVal, prodCol, prodRow,
	consTileVal, consCol, consRow, consIdx);
	if (!ok) {
	// Step 4: all modes exhausted — emit a hard error.
	state.deviceOp.emitError(
	llvm::Twine("conduit-to-dma: no DMA resources available for "
	"conduit '") +
	name + "': circuit DMA MM2S channels exhausted on tile (" +
	llvm::Twine(prodCol) + "," + llvm::Twine(prodRow) +
	") and packet DMA fallback is also ineligible "
	"(check BD budget, lock budget, and packet flow ID budget)");
	// B-3 fix: return immediately so the outer conduit loop does not
	// continue processing subsequent conduits with broken state after
	// both circuit DMA and packet fallback have been exhausted.
	state.passFailed = true;
	return;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 1089 to 1092 in 39c4268

	// tryPacketFallback records conduitMM2SChannel and
	// conduitConsS2MMChannel internally; skip the circuit path below.
	continue;
	}

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 1094 to 1102 in 39c4268

	// S2MM fuse group: reuse existing S2MM channel if another conduit
	// in the same fuse group already allocated one on this tile.
	int32_t s2mmChannel;
	if (!info.fuseGroupS2MM.empty()) {
	std::string qS2MM = state.qualifyFuseGroup(info.fuseGroupS2MM,
	info.deviceIndex);
	auto it = state.fuseGroupS2MMChannel.find(qS2MM);
	if (it != state.fuseGroupS2MMChannel.end()) {
	s2mmChannel = it->second;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 1104 in 39c4268

uint32_t maxS2MM_4c = 2;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 1116 to 1117 in 39c4268

	llvm::Twine(consCol) + "," + llvm::Twine(consRow) +
	"): all " + llvm::Twine(maxS2MM_4c) + " channels in use");

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 1122 to 1143 in 39c4268

	state.fuseGroupS2MMChannel[qS2MM] = s2mmChannel;
	}
	std::string bdKey = name + "__s2mm_" + std::to_string(consIdx);
	state.fuseGroupMembers[qS2MM].push_back(bdKey);
	} else {
	// Bounds-check S2MM channels on the consumer tile.
	uint32_t maxS2MM_4c = 2;
	if (state.targetModel)
	maxS2MM_4c = state.targetModel->getNumDestSwitchboxConnections(
	static_cast<int>(consCol), static_cast<int>(consRow),
	AIE::WireBundle::DMA);
	int32_t nextS2MM_4c = state.tileNextS2MMChannel.count(consTileVal)
	? state.tileNextS2MMChannel[consTileVal]
	: 0;
	if (static_cast<uint32_t>(nextS2MM_4c) >= maxS2MM_4c) {
	state.deviceOp.emitError(
	llvm::Twine("conduit-to-dma: S2MM DMA channel exhausted on "
	"tile (") +
	llvm::Twine(consCol) + "," + llvm::Twine(consRow) +
	"): all " + llvm::Twine(maxS2MM_4c) + " channels in use");
	state.passFailed = true;
	return;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 1145 to 1157 in 39c4268

	s2mmChannel = state.tileNextS2MMChannel[consTileVal]++;
	}
	state.conduitConsS2MMChannel[{name, consIdx}] = s2mmChannel;
	// Deduplicate: skip if the same source→dest port pair was already
	// emitted by the packet broadcast path or a fuse group partner.
	// This prevents duplicate packet_flow + aie.flow for the same ports
	// when MM2S fuse group members share the same consumer S2MM channel.
	FlowKey fk{prodTileVal.getAsOpaquePointer(), mm2sChannel,
	consTileVal.getAsOpaquePointer(), s2mmChannel};
	if (emittedFlowPorts.count(fk))
	continue;
	emittedFlowPorts.insert(fk);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 1159 to 1161 in 39c4268

	state.emitFlow(info.routingMode, prodTileVal, AIE::WireBundle::DMA,
	mm2sChannel, consTileVal, AIE::WireBundle::DMA,
	s2mmChannel);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 1163 in 39c4268

}

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ObjectFifoToConduit.cpp

Lines 1053 to 1054 in 39c4268

	loc, elemTy,
	mlir::FlatSymbolRefAttr::get(ctx, name));

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ObjectFifoToConduit.cpp

Lines 1271 to 1272 in 39c4268

	loc, storedVal,
	mlir::FlatSymbolRefAttr::get(ctx, name));

[github-actions]

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

clang-format

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 1232 to 1233 in 27b72c1

	if (ConduitInfo *dstInfo =
	state.lookupConduit(dstName2, linkOp.op)) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 1856 to 1857 in 27b72c1

	std::string qFG = state.qualifyFuseGroup(info.fuseGroup,
	info.deviceIndex);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 1916 to 1917 in 27b72c1

	bdTermBlock =
	(info.dmaRepeat > 0) ? addBlock() : nullptr;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 2382 to 2383 in 27b72c1

	// first conduit emits a dma_start. Subsequent conduits append BD blocks
	// and the post-pass links all BD chains into a single combined ring.

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Line 2398 in 27b72c1

std::to_string(consRow) + "_ch" + std::to_string(s2mmChannel),

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 2439 to 2440 in 27b72c1

	// Linear chain condition: either dma_repeat>0 (finite DMA task queue),
	// or putCount>1 with no dmaRepeat (N sequential puts merged by

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 2442 to 2444 in 27b72c1

	isLinearChain =
	(info.dmaRepeat > 0) ||
	(info.putCount > 1 && info.dmaRepeat == 0);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 2508 to 2509 in 27b72c1

	builder.create<AIE::NextBDOp>(state.deviceOp.getLoc(),
	bdBlocks[0]);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 2529 to 2530 in 27b72c1

	std::string bdKey =
	name + "__s2mm_" + std::to_string(consIdx);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALink.cpp

Lines 2537 to 2540 in 27b72c1

	std::string bdKey =
	name + "__s2mm_" + std::to_string(consIdx);
	std::string qS2MM = state.qualifyFuseGroup(info.fuseGroupS2MM,
	info.deviceIndex);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 729 to 730 in 27b72c1

	auto ctrKey =
	std::make_tuple(conduitName.str(), col, row, false);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 741 to 742 in 27b72c1

	initBuilder.create<mlir::arith::ConstantIndexOp>(
	loc, rotationBufSlot);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 755 to 756 in 27b72c1

	auto ctrKey =
	std::make_tuple(conduitName.str(), col, row, true);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 760 to 761 in 27b72c1

	initBuilder.setInsertionPointAfterValue(
	resolvedProducerRotationBuf);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 770 to 772 in 27b72c1

	initBuilder.create<mlir::memref::StoreOp>(
	loc, zero, resolvedProducerRotationBuf,
	mlir::ValueRange{slotIdx});

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1044 to 1047 in 27b72c1

	// This is the reverse of --dma-task-to-conduit. After --conduit-fuse-operators
	// merges runtime_sequences and eliminates dead block args, the remaining
	// put/get ops correspond 1:1 (positionally) to the runtime_sequence block
	// args. Each Nth conduit.put_memref/get_memref maps to block arg N.

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1063 to 1064 in 27b72c1

	if (auto cc = alloc->getAttrOfType<mlir::FlatSymbolRefAttr>(
	"conduit_channel"))

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1105 to 1106 in 27b72c1

	bool isS2MM =
	(conduitToDir[conduitName] == AIE::DMAChannelDir::S2MM);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1117 to 1119 in 27b72c1

	if (auto dimsAttr =
	op->getAttrOfType<AIE::BDDimLayoutArrayAttr>(
	"producer_dimensions"))

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1126 to 1129 in 27b72c1

	mlir::OperationState configState(loc,
	"aiex.dma_configure_task_for");
	configState.addAttribute(
	"alloc", mlir::FlatSymbolRefAttr::get(ctx, allocSym));

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1131 to 1132 in 27b72c1

	configState.addAttribute("issue_token",
	builder.getBoolAttr(true));

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1144 to 1146 in 27b72c1

	builder.create<AIE::DMABDOp>(
	loc, bufArg, /*offset=*/0,
	static_cast<int>(numElems), dims);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1148 to 1150 in 27b72c1

	builder.create<AIE::DMABDOp>(
	loc, bufArg, /*offset=*/0,
	static_cast<int>(numElems));

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1152 to 1153 in 27b72c1

	bdBlock->back().setAttr("burst_length",
	builder.getI32IntegerAttr(0));

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1175 to 1176 in 27b72c1

	mlir::OperationState awaitState(rtSeq.getLoc(),
	"aiex.dma_await_task");

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMALower.cpp

Lines 1183 to 1184 in 27b72c1

	mlir::OperationState freeState(rtSeq.getLoc(),
	"aiex.dma_free_task");

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMAPass.cpp

Lines 330 to 339 in 27b72c1

	// Skip cores that already have link_with or link_files set.
	if (coreOp.getLinkWith() || coreOp.getLinkFiles())
	return;
	// Check if any func.call inside this core references a function
	// with link_with.
	std::string linkWithValue;
	coreOp.walk([&](mlir::func::CallOp callOp) {
	auto it = funcLinkWith.find(callOp.getCallee());
	if (it != funcLinkWith.end() && linkWithValue.empty())
	linkWithValue = it->second;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMAPass.cpp

Lines 341 to 344 in 27b72c1

	if (!linkWithValue.empty())
	coreOp.setLinkWithAttr(
	mlir::StringAttr::get(module.getContext(), linkWithValue));
	});

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 249 in 27b72c1

locks.second.getResult()};

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 434 to 435 in 27b72c1

	if (!isAIE2 && !info.externalBuffers.empty() &&
	!info.noLocks) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 484 to 485 in 27b72c1

	std::string qS2MM = state.qualifyFuseGroup(info.fuseGroupS2MM,
	info.deviceIndex);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 529 to 530 in 27b72c1

	llvm::Twine(consCol) + "," + llvm::Twine(consRow) +
	"): all " + llvm::Twine(maxS2MM_4a) + " channels in use");

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 726 in 27b72c1

llvm::StringMap<unsigned> fuseGroupPacketIDNext; // qFG → next member index

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 783 to 786 in 27b72c1

	for (auto &[name, info] : state.conduitMap) {
	// Multi-device: ensure tile lookups target the correct device.
	if (state.isMultiDevice())
	state.switchToDeviceIndex(info.deviceIndex);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 788 to 790 in 27b72c1

	// Pass 0: only packet conduits. Pass 1: everything else.
	if (flowPass == 0 && info.routingMode != "packet") continue;
	if (flowPass == 1 && info.routingMode == "packet") continue;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 792 to 807 in 27b72c1

	if (info.routingMode == "cascade")
	continue;
	if (info.sharedMemory)
	continue;
	if (state.linkSrcNamesEarly.count(name) ||
	state.linkJoinSrcNames.count(name))
	continue;
	// Link destinations: flows are emitted by linkPhase() — skip here to
	// avoid duplicate flows.
	if (state.linkDstNames.count(name))
	continue;
	auto [prodCol, prodRow] = info.producerTileCoord;
	if (prodCol < 0 || prodRow == 0)
	continue;
	if (info.consumerTileCoords.empty())
	continue;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 809 to 812 in 27b72c1

	AIE::TileOp prodTile = state.lookupTileByCoord(prodCol, prodRow);
	if (!prodTile)
	continue;
	mlir::Value prodTileVal = prodTile.getResult();

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 814 to 815 in 27b72c1

	if (!info.consumerTileBuffers.count(prodTileVal))
	continue;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 817 in 27b72c1

builder.setInsertionPoint(state.deviceBody->getTerminator());

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 819 to 825 in 27b72c1

	// ---- Determine hardware MM2S channel count for this tile. ----
	// Used by the mode=any exhaustion check (Step 3.5).
	uint32_t maxMM2S = 2; // hardware default: 2 MM2S per compute tile
	if (state.targetModel)
	maxMM2S = state.targetModel->getNumSourceSwitchboxConnections(
	static_cast<int>(prodCol), static_cast<int>(prodRow),
	AIE::WireBundle::DMA);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 827 to 837 in 27b72c1

	// ---- Assign MM2S channel (fused groups share a channel). ----
	// Check if Phase 4b already assigned an MM2S channel for this conduit
	// (happens when the conduit has both compute and shim consumers — the
	// shim consumer flow and the compute consumer flow share the same
	// producer-side MM2S channel as a hardware broadcast).
	int32_t mm2sChannel = -1;
	bool usedPacketFallback = false;
	{
	auto existingIt = state.conduitMM2SChannel.find(name);
	if (existingIt != state.conduitMM2SChannel.end()) {
	mm2sChannel = existingIt->second;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 839 in 27b72c1

}

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 841 to 876 in 27b72c1

	if (mm2sChannel >= 0) {
	// Already assigned by Phase 4b — reuse (broadcast from same MM2S port).
	} else if (!info.fuseGroup.empty()) {
	std::string qFG = state.qualifyFuseGroup(info.fuseGroup,
	info.deviceIndex);
	auto it = state.fuseGroupMM2SChannel.find(qFG);
	if (it != state.fuseGroupMM2SChannel.end()) {
	mm2sChannel = it->second;
	} else {
	mm2sChannel = state.tileNextMM2SChannel[prodTileVal]++;
	state.fuseGroupMM2SChannel[qFG] = mm2sChannel;
	}
	state.fuseGroupMembers[qFG].push_back(name);
	state.conduitMM2SChannel[name] = mm2sChannel;
	} else if (info.routingMode == "any") {
	// mode=any: check whether a circuit DMA channel is available.
	int32_t nextCh = state.tileNextMM2SChannel.count(prodTileVal)
	? state.tileNextMM2SChannel[prodTileVal]
	: 0;
	if (static_cast<uint32_t>(nextCh) < maxMM2S) {
	// A free circuit-mode channel exists — use it.
	mm2sChannel = state.tileNextMM2SChannel[prodTileVal]++;
	state.conduitMM2SChannel[name] = mm2sChannel;
	} else {
	// Circuit DMA exhausted; flag that Step 3.5 handles emission below.
	usedPacketFallback = true;
	}
	} else {
	// Check if a circuit DMA channel is available before allocating.
	// If exhausted, fall back to packet-switched DMA for non-cascade/
	// non-shared-memory channels (extends the mode=any fallback).
	int32_t nextCh = state.tileNextMM2SChannel.count(prodTileVal)
	? state.tileNextMM2SChannel[prodTileVal]
	: 0;
	if (static_cast<uint32_t>(nextCh) < maxMM2S) {
	mm2sChannel = state.tileNextMM2SChannel[prodTileVal]++;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 878 to 884 in 27b72c1

	// Track packet-mode channel designation for Step 3.5c.
	if (info.routingMode == "packet" && prodTile) {
	auto key = std::make_pair(prodTile.getOperation(),
	static_cast<int>(mm2sChannel));
	state.pktChannelState.isPacketChannel[key] = true;
	// NOTE: usedPacketFallback is NOT set here; explicit packet-mode
	// broadcast is handled below (single multi-dest packet flow).

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 886 to 893 in 27b72c1

	} else if (info.routingMode != "cascade" &&
	info.routingMode != "shared_memory" &&
	info.routingMode != "stream") {
	// Circuit DMA exhausted on producer tile — fall back to
	// packet-switched DMA regardless of explicit routing_mode.
	// tryPacketFallback (Step 3.5c) will reuse an existing
	// packet-designated MM2S channel if one exists.
	usedPacketFallback = true;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 895 to 897 in 27b72c1

	// Cascade/shared_memory/stream cannot use packet fallback.
	mm2sChannel = state.tileNextMM2SChannel[prodTileVal]++;
	state.conduitMM2SChannel[name] = mm2sChannel;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 899 in 27b72c1

}

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 901 to 924 in 27b72c1

	// ---- Explicit packet-mode broadcast: single multi-dest flow. ----
	// For routing_mode="packet", emit one aie.packet_flow with all consumer
	// destinations and a single packet ID. The switchbox hardware broadcasts
	// each packet to all destinations. This matches the oracle's behavior
	// (AIEObjectFifoStatefulTransform) where one bdPacket ID is used for all
	// producer MM2S BDs and one packet_flow carries multiple packet_dest ops.
	if (info.routingMode == "packet" && mm2sChannel >= 0 &&
	!info.consumerTileCoords.empty()) {
	if (!state.packetIDAllocator) {
	state.module.emitError(
	"internal error: packetIDAllocator not initialized");
	state.passFailed = true;
	return;
	}
	// Use pre-allocated aligned block ID if this channel belongs to a fuse
	// group with multiple packet members; otherwise fall back to sequential.
	std::optional<uint8_t> pktID;
	std::string qFG;
	if (!info.fuseGroup.empty()) {
	qFG = state.qualifyFuseGroup(info.fuseGroup, info.deviceIndex);
	auto baseIt = fuseGroupPacketIDBase.find(qFG);
	if (baseIt != fuseGroupPacketIDBase.end()) {
	unsigned idx = fuseGroupPacketIDNext[qFG]++;
	pktID = static_cast<uint8_t>(baseIt->second + idx);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 926 in 27b72c1

}

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 927 to 946 in 27b72c1

	if (!pktID) {
	mlir::Value pktDomain = state.getMemTileDomain(prodTileVal);
	pktID = state.packetIDAllocator->allocate(pktDomain);
	}
	if (!pktID) {
	state.passFailed = true;
	return;
	}
	state.conduitPacketID[name] = *pktID;
	auto pktFlow = builder.create<AIE::PacketFlowOp>(
	state.deviceOp.getLoc(), static_cast<int8_t>(*pktID),
	/*keep_pkt_header=*/mlir::BoolAttr{},
	/*priority_route=*/mlir::BoolAttr{});
	mlir::Region &region = pktFlow.getPorts();
	mlir::Block *pktBlock = builder.createBlock(&region);
	builder.setInsertionPointToStart(pktBlock);
	builder.create<AIE::PacketSourceOp>(state.deviceOp.getLoc(), prodTileVal,
	AIE::WireBundle::DMA,
	static_cast<int32_t>(mm2sChannel));

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 948 in 27b72c1

for (unsigned consIdx = 0; consIdx < info.consumerTileCoords.size();

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 954 in 27b72c1

AIE::TileOp consTile = state.lookupTileByCoord(consCol, consRow);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 957 in 27b72c1

mlir::Value consTileVal = consTile.getResult();

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 959 to 988 in 27b72c1

	// Allocate S2MM channel on the consumer tile.
	// Only share S2MM ports between packet channels that have the same
	// dma_channel_group. Independent packet channels (no group) get
	// separate S2MM ports to prevent data crossover (e.g., Q/K vs V in
	// flash attention — same tile, different data).
	// Effective group key: dma_channel_group_s2mm if set, else
	// dma_channel_group.
	std::string s2mmGrp = state.qualifyFuseGroup(
	!info.fuseGroupS2MM.empty() ? info.fuseGroupS2MM
	: info.fuseGroup,
	info.deviceIndex);
	int32_t s2mmChannel;
	bool s2mmShared = false;
	if (!s2mmGrp.empty()) {
	auto it = state.fuseGroupS2MMChannel.find(s2mmGrp);
	if (it != state.fuseGroupS2MMChannel.end()) {
	s2mmChannel = it->second;
	s2mmShared = true;
	}
	}
	if (!s2mmShared) {
	uint32_t maxS2MM_pkt = 2;
	if (state.targetModel)
	maxS2MM_pkt = state.targetModel->getNumDestSwitchboxConnections(
	static_cast<int>(consCol), static_cast<int>(consRow),
	AIE::WireBundle::DMA);
	int32_t nextS2MM_pkt = state.tileNextS2MMChannel.count(consTileVal)
	? state.tileNextS2MMChannel[consTileVal]
	: 0;
	if (static_cast<uint32_t>(nextS2MM_pkt) >= maxS2MM_pkt) {

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 990 to 993 in 27b72c1

	llvm::Twine("conduit-to-dma: S2MM DMA channel exhausted on "
	"tile (") +
	llvm::Twine(consCol) + "," + llvm::Twine(consRow) +
	"): all " + llvm::Twine(maxS2MM_pkt) + " channels in use");

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 997 to 1059 in 27b72c1

	s2mmChannel = state.tileNextS2MMChannel[consTileVal]++;
	if (!s2mmGrp.empty())
	state.fuseGroupS2MMChannel[s2mmGrp] = s2mmChannel;
	}
	state.conduitConsS2MMChannel[{name, consIdx}] = s2mmChannel;
	// Lock sharing: only share locks when channels share an S2MM port
	// (same dma_channel_group). Independent channels use separate locks.
	if (s2mmShared) {
	auto lockIt = state.pktTileS2MMLock.find(consTileVal);
	if (lockIt != state.pktTileS2MMLock.end()) {
	info.consumerTileLocks[consTileVal] = {
	lockIt->second.first.getDefiningOp<AIE::LockOp>(),
	lockIt->second.second.getDefiningOp<AIE::LockOp>()};
	}
	} else if (!s2mmGrp.empty()) {
	// First in group: record locks for future group members.
	auto &locks = info.consumerTileLocks[consTileVal];
	if (locks.first && locks.second) {
	state.pktTileS2MMLock[consTileVal] = {locks.first.getResult(),
	locks.second.getResult()};
	}
	}
	builder.create<AIE::PacketDestOp>(state.deviceOp.getLoc(), consTileVal,
	AIE::WireBundle::DMA,
	static_cast<int32_t>(s2mmChannel));
	// Record the emitted port pair so that fuse group partners sharing
	// the same MM2S+S2MM ports do not emit a duplicate circuit flow.
	FlowKey fk{prodTileVal.getAsOpaquePointer(), mm2sChannel,
	consTileVal.getAsOpaquePointer(), s2mmChannel};
	emittedFlowPorts.insert(fk);
	}
	builder.create<AIE::EndOp>(state.deviceOp.getLoc());
	builder.setInsertionPointAfter(pktFlow);
	continue; // skip per-consumer circuit/fallback flow loop
	}
	// ---- Emit flows per consumer. ----
	for (unsigned consIdx = 0; consIdx < info.consumerTileCoords.size();
	++consIdx) {
	auto [consCol, consRow] = info.consumerTileCoords[consIdx];
	if (consRow == 0)
	continue;
	// For single-consumer conduits, adjacent tiles use shared memory
	// (Phase 3c) — no DMA flow needed. For broadcast (multi-consumer),
	// Phase 3c is skipped; all consumers use DMA, so flows are needed
	// for every consumer regardless of adjacency.
	// Exception: forceDMA forces DMA even for adjacent tiles.
	// Also: when shim consumers exist, the producer needs DMA MM2S
	// regardless (to reach the shim tile via the switchbox network),
	// so the compute consumer flow must also be emitted.
	if (!info.forceDMA && info.consumerTileCoords.size() == 1 &&
	info.shimConsumerTileCoords.empty()) {
	bool explicitSharedMem = (info.routingMode == "shared_memory");
	bool rightAdj = state.targetModel->isLegalMemAffinity(prodCol, prodRow,
	consCol, consRow);
	bool leftAdj = state.targetModel->isLegalMemAffinity(consCol, consRow,
	prodCol, prodRow);
	if (explicitSharedMem || rightAdj || leftAdj)

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 1061 to 1087 in 27b72c1

	}
	AIE::TileOp consTile = state.lookupTileByCoord(consCol, consRow);
	if (!consTile)
	continue;
	mlir::Value consTileVal = consTile.getResult();
	if (usedPacketFallback) {
	// Step 3.5: attempt packet DMA fallback.
	bool ok =
	tryPacketFallback(state, name, info, prodTileVal, prodCol, prodRow,
	consTileVal, consCol, consRow, consIdx);
	if (!ok) {
	// Step 4: all modes exhausted — emit a hard error.
	state.deviceOp.emitError(
	llvm::Twine("conduit-to-dma: no DMA resources available for "
	"conduit '") +
	name + "': circuit DMA MM2S channels exhausted on tile (" +
	llvm::Twine(prodCol) + "," + llvm::Twine(prodRow) +
	") and packet DMA fallback is also ineligible "
	"(check BD budget, lock budget, and packet flow ID budget)");
	// B-3 fix: return immediately so the outer conduit loop does not
	// continue processing subsequent conduits with broken state after
	// both circuit DMA and packet fallback have been exhausted.
	state.passFailed = true;
	return;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 1089 to 1092 in 27b72c1

	// tryPacketFallback records conduitMM2SChannel and
	// conduitConsS2MMChannel internally; skip the circuit path below.
	continue;
	}

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 1094 to 1102 in 27b72c1

	// S2MM fuse group: reuse existing S2MM channel if another conduit
	// in the same fuse group already allocated one on this tile.
	int32_t s2mmChannel;
	if (!info.fuseGroupS2MM.empty()) {
	std::string qS2MM = state.qualifyFuseGroup(info.fuseGroupS2MM,
	info.deviceIndex);
	auto it = state.fuseGroupS2MMChannel.find(qS2MM);
	if (it != state.fuseGroupS2MMChannel.end()) {
	s2mmChannel = it->second;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 1104 in 27b72c1

uint32_t maxS2MM_4c = 2;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 1116 to 1117 in 27b72c1

	llvm::Twine(consCol) + "," + llvm::Twine(consRow) +
	"): all " + llvm::Twine(maxS2MM_4c) + " channels in use");

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 1122 to 1143 in 27b72c1

	state.fuseGroupS2MMChannel[qS2MM] = s2mmChannel;
	}
	std::string bdKey = name + "__s2mm_" + std::to_string(consIdx);
	state.fuseGroupMembers[qS2MM].push_back(bdKey);
	} else {
	// Bounds-check S2MM channels on the consumer tile.
	uint32_t maxS2MM_4c = 2;
	if (state.targetModel)
	maxS2MM_4c = state.targetModel->getNumDestSwitchboxConnections(
	static_cast<int>(consCol), static_cast<int>(consRow),
	AIE::WireBundle::DMA);
	int32_t nextS2MM_4c = state.tileNextS2MMChannel.count(consTileVal)
	? state.tileNextS2MMChannel[consTileVal]
	: 0;
	if (static_cast<uint32_t>(nextS2MM_4c) >= maxS2MM_4c) {
	state.deviceOp.emitError(
	llvm::Twine("conduit-to-dma: S2MM DMA channel exhausted on "
	"tile (") +
	llvm::Twine(consCol) + "," + llvm::Twine(consRow) +
	"): all " + llvm::Twine(maxS2MM_4c) + " channels in use");
	state.passFailed = true;
	return;

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 1145 to 1157 in 27b72c1

	s2mmChannel = state.tileNextS2MMChannel[consTileVal]++;
	}
	state.conduitConsS2MMChannel[{name, consIdx}] = s2mmChannel;
	// Deduplicate: skip if the same source→dest port pair was already
	// emitted by the packet broadcast path or a fuse group partner.
	// This prevents duplicate packet_flow + aie.flow for the same ports
	// when MM2S fuse group members share the same consumer S2MM channel.
	FlowKey fk{prodTileVal.getAsOpaquePointer(), mm2sChannel,
	consTileVal.getAsOpaquePointer(), s2mmChannel};
	if (emittedFlowPorts.count(fk))
	continue;
	emittedFlowPorts.insert(fk);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Lines 1159 to 1161 in 27b72c1

	state.emitFlow(info.routingMode, prodTileVal, AIE::WireBundle::DMA,
	mm2sChannel, consTileVal, AIE::WireBundle::DMA,
	s2mmChannel);

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ConduitToDMARoute.cpp

Line 1163 in 27b72c1

}

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ObjectFifoToConduit.cpp

Lines 1053 to 1054 in 27b72c1

	loc, elemTy,
	mlir::FlatSymbolRefAttr::get(ctx, name));

---

[clang-format] _{reported by reviewdog 🐶}

mlir-aie/lib/Dialect/Conduit/Transforms/ObjectFifoToConduit.cpp

Lines 1271 to 1272 in 27b72c1

	loc, storedVal,
	mlir::FlatSymbolRefAttr::get(ctx, name));