[PATCH] D76019: [mlir][NFC] Move the definition of AffineApplyOp to ODS

Jacques Pienaar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 12 10:51:15 PDT 2020


jpienaar accepted this revision.
jpienaar marked an inline comment as done.
jpienaar added a comment.
This revision is now accepted and ready to land.

Looks good, keeping it mostly mechanical is good :)



================
Comment at: mlir/include/mlir/Dialect/AffineOps/AffineOps.td:87
+    /// Returns true if the result of this operation is a symbol.
+    bool isValidSymbol();
+
----------------
These could be derived attributes :-)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76019





More information about the llvm-commits mailing list