Remove (X, Y) coordinates from NpuDmaMemcpyNdOp

[pvasireddy-amd]

Issue #1964
Requires #1997

This PR fixes an issue where the X and Y input attributes of the NpuDmaMemcpyNdOp given through the python bindings were always (0, 0), which may not be the coordinates of the Shim tile that the operation will be mapped to (and were not intended to represent that, by design). Instead, the lowering uses the metadata input of the NpuDmaMemcpyNdOp to find the corresponding ShimDMAAllocationOp. As this operation has the column coordinate (and the row coordinate is implicit for Shim tiles) this PR removes the X and Y coordinates from the NpuDmaMemcpyNdOp altogether and fully relies on the metadata to find the information.

[github-actions]

Coverage Report

Created: 2024-12-16 17:03

Click here for information about interpreting this report.

Filename	Function Coverage	Line Coverage	Region Coverage	Branch Coverage
Totals	-	-	-	-

Generated by llvm-cov -- llvm version 14.0.0

[pvasireddy-amd]

Debug status:

1. Error occurs when changing the device name from npu1_1col to npu1 while trying to perform a passthrough from ShimTile (0,0) to ComputeTile (0,2). This results in the following error: aiex.npu.dma_memcpy_nd' op Unsupported tile type at (0,0). Must be ShimNOC, Mem, or Core. This happens because getX() and getY() are used instead of the column and row information, which should be obtained from the metadata of dma_memcpy_nd in this line. Tile (0,0) is considered a ShimPLTile when the entire NPU is used, but this check needs to be moved since the row and column information is not available at the NpuDmaMemcpyNd stage.
2. The second case involves changing both the ShimTile and ComputeTile to columns other than the zeroth column and using the appropriate device name for the same passthrough example. This results in a runtime error. One temporary workaround is to declare at least one tile (which can be either a ShimTile, MemTile, or ComputeTile with an empty body) in the zeroth column, and the test passes. The reason for this dependency on the zeroth column is unclear.