address comments

Author: chencha3

mlir/include/mlir/Dialect/XeGPU/IR/XeGPUDialect.td

@@ -38,8 +38,6 @@ def XeGPU_Dialect : Dialect {
     let useDefaultAttributePrinterParser = true;
 
     let extraClassDeclaration = [{
-      static constexpr const char *operandLayoutNamePrefix = "layout_operand_";
-      static constexpr const char *resultLayoutNamePrefix = "layout_result_";
       /// Checks if the given shape can be evenly distributed based on the layout
       /// and data factors provided by the LayoutAttr.
       static bool isEvenlyDistributable(llvm::ArrayRef<int64_t> shape, xegpu::LayoutAttr attr);

mlir/lib/Dialect/XeGPU/Transforms/XeGPUUnroll.cpp

@@ -446,10 +446,8 @@ struct UnrollDpasOp : public UnrollPattern<xegpu::DpasOp> {
     if (c)
       cVals = packWrapper(c, cBlockSize);
 
-    // Vals are empty due to invalid blocking size, or with size 1 due to
-    // the original shape is the same with the blocking size. The op will
-    // be skipped if every operand got an invalid blocking size or the
-    // original shape is the same with the blocking size.
+    // skip the operation if every operand has an invalid blocking size (empty)
+    // or if the original shape matches the blocking size (size == 1).
     if (aVals.size() <= 1 && bVals.size() <= 1 && cVals.size() <= 1)
       return failure();