Skip to content

Commit 9c63091

Browse files
hunhoffeclaude
andcommitted
[dyn-seq] dma_op.py: update to renamed transfer_len kwarg + static len form
The self-review commit renamed the dma_bd() wrapper's len kwarg to transfer_len (avoids shadowing the builtin, matches shim_dma_bd) and made a constant len land in the static_len attribute rather than an arith.constant operand. This test still passed len= and CHECK'd the operand form, so it hit "got multiple values for keyword argument 'len'". Route through transfer_len and expect the bare `len = 2`. Co-Authored-By: Claude <noreply@anthropic.com>
1 parent e9456b4 commit 9c63091

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

test/python/dma_op.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,8 +36,7 @@
3636
# %memtile_dma_0_1 = aie.memtile_dma(%tile_0_1) {
3737
# %0 = aie.dma(MM2S, 0) [{
3838
# aie.use_lock(%lock_0_1_0, AcquireGreaterEqual)
39-
# %c2_i32 = arith.constant 2 : i32
40-
# aie.dma_bd(%mem_A : memref<2x1xi16> len = %c2_i32 sizes = [1, 1, 1, 1] strides = [1, 1, 1, 1])
39+
# aie.dma_bd(%mem_A : memref<2x1xi16> len = 2 sizes = [1, 1, 1, 1] strides = [1, 1, 1, 1])
4140
# aie.use_lock(%lock_0_1, Release)
4241
# }]
4342
# aie.end
@@ -70,7 +69,7 @@ def dma_in_A_to_compute():
7069
use_lock(cons_lock_mem_A, LockAction.AcquireGreaterEqual)
7170
dma_bd(
7271
buff_mem_A,
73-
len=2,
72+
transfer_len=2,
7473
sizes=[1, 1, 1, 1],
7574
strides=[1, 1, 1, 1],
7675
)

0 commit comments

Comments
 (0)