[PATCH] D73213: [mlir] [VectorOps] Implement vector tuple get folding

[Andy Davis via Phabricator via llvm-commits]

andydavis1 added a comment.

Looks good.  Thanks Aart!



================
Comment at: mlir/lib/Dialect/VectorOps/VectorOps.cpp:1696
+    // into:
+    //    %t = vector.tuple .., %e_i, ..  // one less use
+    //    %x = %e_i
----------------
I see. So the idea is to rewrite users one at a time, and eventually the tuple op will have no more users and get canonicalized away?


================
Comment at: mlir/test/Dialect/VectorOps/vector-transforms.mlir:306
+
+// CHECK-LABEL: func @tuple_get(%arg0: vector<4xf32>, %arg1: vector<8xf32>)
+//       CHECK: return %arg1
----------------
Is it worth adding a test which folds away all of the tuple op users, and test that the tuple op gets canonicalized away?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73213/new/

https://reviews.llvm.org/D73213