[Mlir-commits] [mlir] [MLIR][XeGPU] Recover temporary layout from Anchor Layout (PR #191947)

[Jianhui Li]

Jianhui-Li wrote:

> LGTM.
> 
> I suggest removing the dummy conv_layout op added in unit tests. Instead write the layout recovered test and avoid running layout recovery for distribution unit tests (using a pass option). This is quite common in MLIR.
> 
> For testing layout recovery, we definitely need a test pass rather than relying on exiting tests. This will help us to closely test the recovery process rather than replying on existing indirect tests.
> 
> This will keep the tests simple and maintainable.

added test pass for layout recovery. 

https://github.com/llvm/llvm-project/pull/191947