[Mlir-commits] [mlir] [mlir][XeGPU] Add MemoryEffectsOpInterface for XeGPU memory related ops. (PR #125314)

Adam Siemieniuk llvmlistbot at llvm.org
Mon Feb 3 09:46:05 PST 2025


================
@@ -276,7 +276,10 @@ def XeGPU_PrefetchNdOp : XeGPU_Op<"prefetch_nd", []> {
 }
 
 
-def XeGPU_LoadNdOp : XeGPU_Op<"load_nd", [AllElementTypesMatch<["value", "TensorDesc"]>]> {
+def XeGPU_LoadNdOp : XeGPU_Op<"load_nd", [
+  AllElementTypesMatch<["value", "TensorDesc"]>,
+  DeclareOpInterfaceMethods<MemoryEffectsOpInterface>
----------------
adam-smnk wrote:

Just FYI, I can see the changes on your fork but not here. Maybe the latest merge didn't go too well?
It's usually easier to rebase feature branch on top of main and force push to PR.

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


More information about the Mlir-commits mailing list