[Mlir-commits] [mlir] [MLIR][XeGPU] make offsets optional for create_nd_tdesc (PR #148335)

Adam Siemieniuk llvmlistbot at llvm.org
Thu Jul 17 09:22:06 PDT 2025


================
@@ -172,10 +202,16 @@ def XeGPU_CreateNdDescOp: XeGPU_Op<"create_nd_tdesc", [Pure, ViewLikeOpInterface
     /// and `const_shape` will be used to represent the shape of
     /// source operand. They overide static shape from source memref type.
     ArrayRef<int64_t> getStaticSizes() {
+      /// To be compatible with OffsetSizeAndStrideOpInterface, which expects valid return value and perform checks
+      static  llvm::SmallVector<int64_t, 4> emptyShape;
----------------
adam-smnk wrote:

At this point, the op breaks interface contract.
I hope it can be solved with sane defaults - no values provided means zero offsets. Otherwise, we need to drop the interface. 

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


More information about the Mlir-commits mailing list