[Mlir-commits] [mlir] [MLIR][XeVM] Update XeVM prefetch ops. (PR #166445)

[Adam Siemieniuk]

adam-smnk wrote:

Thanks for the example, now I see the issue.

I quickly checked on an example using only `xevm.prefetch` which originally doesn't define any side effects.
Ops without known side effects are generally not eliminated due to conservative evaluation. Same applies to the XeGPU prefetch ops that also don't define side effects.
As such, I think assigning write side effect might be strong "guard" against unwanted modifications.

Could you add a small test case just to avoid regression in the future?

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