nod-ai / iree-amd-aie

IREE plugin repository for the AMD AIE accelerator
Apache License 2.0
46 stars 23 forks source link

Cleanup `amdaie-objectFifo-stateful-transform` #510

Open makslevental opened 4 days ago

makslevental commented 4 days ago

This is a refactor prior to the follow-up removal of this and other passes (in order to reduce the diff in the follow-up). We remove aie.shim_dma support from the pass and XFAILtests that are legacy/erroneous from mlir-aie. E.g., aie.device(npu1_4col) does not support aie.shim_dma (recall we use runtime instructions to program shim DMAs on IPU/NPU).

Note, there are a lot of lines changed; probably easiest to first skim the entire file as a whole rather than as a diff.

Feel free to nitpick/bikeshed - it'll help me "robustify" the final version in the follow-up.