[PATCH] D76479: [mlir][LLVMIR] Fix fusion for rank-0 tensors

Mahesh Ravishankar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 20 09:44:40 PDT 2020


mravishankar accepted this revision.
mravishankar added inline comments.


================
Comment at: mlir/lib/IR/AffineMap.cpp:260
 
-  return get(numResultDims, numResultSyms, results);
+  return results.empty() ? get(numResultDims, 0, this->getContext())
+                         : get(numResultDims, numResultSyms, results);
----------------
There are going to be so many of such cases. The get method that gets context from the result expression should just be removed IMO


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76479





More information about the llvm-commits mailing list