[PATCH] D141867: [mlir][vector] Share enums with the transform dialect

Mon Jan 16 11:15:59 PST 2023

qcolombet added inline comments.

================
Comment at: mlir/include/mlir/Dialect/Vector/TransformOps/VectorTransformOps.td:54
+    (`split_transfers` `=` $split_transfers^)?
+    (`transpose_lowering` `=` $transpose_lowering^)?
+    attr-dict
----------------
We have to explicitly list all the fields here to have the proper parsing of the enums.

This may have an impact on the supported syntax. I.e., I don't know if `attr-dict` allows to put the attributes in whatever order, but if it does, then we lose this ability here.

================
Comment at: mlir/include/mlir/Dialect/Vector/Transforms/VectorTransformsEnums.h:2
+#ifndef VECTOR_TRANSFORMS_ENUMS
+#define VECTOR_TRANSFORMS_ENUMS
+
----------------
This file is a work around the fact that tablegen'ed enums header are not guarded.
I don't know how we want to move forward here.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141867