minor tweaks

Author: newling

mlir/lib/Dialect/Vector/IR/VectorOps.cpp

@@ -6267,9 +6267,9 @@ OpFoldResult vector::TransposeOp::fold(FoldAdaptor adaptor) {
   // %0 = vector.transpose %arg, [0, 1] : vector<2x2xi8> to vector<2x2xi8>
   // %0 = vector.transpose %arg, [1, 0] : vector<1x1xi8> to vector<1x1xi8>
   //
-  // Example of what NOT to fold:
-  //
+  // Example of what not to fold:
   // %0 = vector.transpose %arg, [1, 0] : vector<2x2xi8> to vector<2x2xi8>
+  //
   if (getSourceVectorType() == getResultVectorType() &&
       isOrderPreserving(*this))
     return getVector();
@@ -6514,8 +6514,9 @@ struct TransposeToShapeCast final
 
 void vector::TransposeOp::getCanonicalizationPatterns(
     RewritePatternSet &results, MLIRContext *context) {
-  results.add<FoldTransposeCreateMask, TransposeFolder, FoldTransposeSplat,
-              FoldTransposeBroadcast, TransposeToShapeCast>(context);
+  results.add<FoldTransposeBroadcast, FoldTransposeCreateMask,
+              FoldTransposeSplat, TransposeFolder, TransposeToShapeCast>(
+      context);
 }
 
 //===----------------------------------------------------------------------===//

mlir/test/Dialect/Vector/canonicalize/vector-shape-cast.mlir

@@ -1,5 +1,7 @@
 // RUN: mlir-opt %s -split-input-file -canonicalize |  FileCheck %s
 
+// This file contains tests where there a vector.shape_cast gets canonicalized, or where a
+// vector.shape_cast is the result of a canonicalization. Not all such tests must live in this file.
 
 // +----------------------------------------
 //  Tests of BroadcastToShapeCast

mlir/test/Dialect/Vector/single-fold.mlir

@@ -35,5 +35,4 @@ func.func @fold_insert_in_single_pass() -> vector<2xf16> {
   // CHECK: arith.constant dense<[0.000000e+00, 2.500000e+00]> : vector<2xf16>
   %0 = vector.insert %c2, %cst [%c1] : f16 into vector<2xf16>
   return %0 : vector<2xf16>
-}
-
+}