[Mlir-commits] [mlir] [MLIR][XeGPU][TransformOps] Add set_op_layout_attr op (PR #166854)

[Tuomas Kärnä]

================
@@ -90,6 +90,38 @@ createLayoutAttr(MLIRContext *ctx, ArrayRef<int32_t> sgLayout,
       /*order=*/nullptr);
 }
 
+/// Generate `xegpu::LayoutAttr` from op mixed layout values.
+DiagnosedSilenceableFailure
+getLayoutAttrFromOperands(transform::TransformRewriter &rewriter,
+                          transform::TransformState &state,
+                          TransformOpInterface transformOp,
----------------
tkarna wrote:

Hmm, I think it's cleanest if the use the same error propagation mechanism for all the ops/utility functions. We you could do something like `emitSilenceableFailure(location)` (where `location` is propagated to the utility func) but using the transform op interface itself seems like the right thing to do. Hence passing in the `transfromOp` is justified.

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