[PATCH] D75586: [MLIR][Affine][NFC] Remove obsolete and ambiguous definitions

Frank Laub via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 4 12:21:44 PST 2020


flaub marked an inline comment as done.
flaub added a comment.

I tried to use this from outside of MLIR core and it ended up not possible to even call the `getFlattendAffineExprs` that doesn't take a 3rd argument due to that being ambiguous with the one defined in `AffineStructures`.



================
Comment at: mlir/include/mlir/IR/AffineExpr.h:82
 
-  template <typename U> bool isa() const;
-  template <typename U> U dyn_cast() const;
-  template <typename U> U dyn_cast_or_null() const;
-  template <typename U> U cast() const;
+  template <typename U>
+  bool isa() const;
----------------
rriddle wrote:
> Note, if you use git-clang-format it should only format the changes that you've made without touching existing code.
Thanks! I didn't know about this and I'll try to use this in my workflow from now on :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75586





More information about the llvm-commits mailing list