[Mlir-commits] [mlir] [MLIR][Linalg] pack, unpack to take memref inputs (PR #129036)

[Han-Chung Wang]

================
@@ -1030,6 +1042,10 @@ static Value getPackOpSourceOrPaddedSource(OpBuilder &builder,
     return input;
   }
 
+  // TODO: Support Memref PackOp. Temporarily return failure.
+  if (!packOp.hasPureTensorSemantics())
+    return packOp.getSource();
----------------
hanhanW wrote:

The code does not agree with the comment. How about returning `Value()` when the op does not have pure tensor semantics? Please also update the function comment to reflect the change.

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