[Mlir-commits] [mlir] [uArch][XeGPU] Add XeGPU uArch definition. (PR #153706)

[Adam Siemieniuk]

https://github.com/adam-smnk commented:

I still have concerns when it comes to wrapping fully static data into runtime structures.
It adds both complexity to the design and runtime overhead.

> The choice of runtime vs static allocations and data structures can be optimized in the subsequent iterations.

Pragmatically speaking, this rarely happens.

Design choices aside, overall info and utilities provided look useful.
The exact APIs can be refined once we start integrating uArch info into passes. Then we'll see what works and what's missing.

It might be better to just take it for a test drive once a general cleanup is done.

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