[PATCH] D72555: [mlir][Linalg] Update the semantics, verifier and test for Linalg with tensors.

[Ahmed S. Taei via Phabricator via llvm-commits]

asaadaldien added inline comments.


================
Comment at: mlir/include/mlir/Dialect/Linalg/IR/LinalgTraits.h:204
+      return getOutputBufferType(i - nInputs()).template cast<ShapedType>();
+    return getOutputTensorTypes()[i - getNumInputsAndOutputBuffers()]
+        .template cast<ShapedType>();
----------------
Do we need to `assert(i - getNumInputsAndOutputBuffers() < getOutputTensorTypes().size())` ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72555