[PATCH] D76700: [MLIR] Introduce full/partial tile separation using if/else

Uday Bondhugula via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 25 23:16:24 PDT 2020


bondhugula marked 3 inline comments as done.
bondhugula added inline comments.


================
Comment at: mlir/lib/Analysis/AffineStructures.cpp:2801
+  // Convert to AffineExpr (tree) form.
+  auto boundExpr = getAffineExprFromFlatForm(bound, numDims - 1, numSyms,
+                                             localExprs, context);
----------------
andydavis1 wrote:
> assert(NumDims >= 1)
Added this at the very top as
`assert(pos < numDims)`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76700





More information about the llvm-commits mailing list