.

Author: ita9naiwa

mlir/lib/Dialect/Linalg/IR/LinalgOps.cpp

@@ -4444,7 +4444,6 @@ static LogicalResult commonVerifierPackAndUnPackOp(OpTy packOrUnPack) {
       !packOrUnPack.hasPureTensorSemantics()) {
     return op->emitError("mixing tensor and buffer semantics is not allowed");
   }
-  bool hasTensorSemantics = packOrUnPack.hasPureTensorSemantics();
 
   // Verify inner_dims_pos and outer_dims_perm.
   ShapedType unpackedType = (std::is_same<OpTy, PackOp>::value)

mlir/lib/Dialect/Linalg/Transforms/DataLayoutPropagation.cpp

@@ -63,14 +63,8 @@ getPackingInfoFromOperand(OpOperand *opOperand, linalg::GenericOp genericOp,
                           OpTy packOrUnPackOp) {
   static_assert(llvm::is_one_of<OpTy, linalg::PackOp, linalg::UnPackOp>::value,
                 "applies to only pack or unpack operations");
-  if constexpr (std::is_same_v<OpTy, linalg::PackOp>) {
-    if (!packOrUnPackOp.hasPureTensorSemantics())
-      return failure();
-  }
-  if constexpr (std::is_same_v<OpTy, linalg::UnPackOp>) {
-    if (!packOrUnPackOp.hasPureTensorSemantics())
-      return failure();
-  }
+  if (!packOrUnPackOp.hasPureTensorSemantics())
+    return failure();
 
   LLVM_DEBUG(
       { llvm::dbgs() << "--- Construct PackInfo From an operand ---\n"; });