[PATCH] D76233: Affine expr simplification for add of const multiple of same expression

Uday Bondhugula via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 16 19:40:28 PDT 2020


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

Thanks for the review.



================
Comment at: mlir/include/mlir/IR/AffineExpr.h:191
   /* implicit */ AffineConstantExpr(AffineExpr::ImplType *ptr);
+  AffineConstantExpr() : AffineConstantExpr(nullptr) {}
   int64_t getValue() const;
----------------
rriddle wrote:
> Can you just add `= nullptr` to the other constructor?
Sure, done. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76233





More information about the llvm-commits mailing list