XRTBackend API Cleanup - Githubissues

Xilinx / mlir-air

MIT License

76 stars 26 forks source link

XRTBackend API Cleanup #634

Closed hunhoffe closed 3 months ago

hunhoffe commented 3 months ago

There are a few things about the XRTBackend that are a bit unintuitive:

You can't actually set the insts file path (unused parameters)
There are unneeded parameters (module in load())
There aren't safety checks for the state that is saved in the backend during load()

This PR cleans up a few aspects of XRTBackend to ensure parameters make sense and are used as one would expect.