It also unifies the computation of StridedLayoutAttr. If the stride is
static known value, we can just use it.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
This addresses the refactoring part about my review comment in https://reviews.llvm.org/D154178
mlir/lib/Dialect/MemRef/Utils/MemRefUtils.cpp | ||
---|---|---|
50 | Didn't look deeply at this method yet but have you tried using/extending utils from mlir/include/mlir/Dialect/Utils/IndexingUtils.h ? Given that I am seeing a lot of linearize / delinearize business in there I thought I would surface. Bonus points to refactoring this enough that it could become a new helper directly in IndexingUtils.h, but this may be a bridge too far, haven't thought about this deeply enough yet. |
Didn't look deeply at this method yet but have you tried using/extending utils from mlir/include/mlir/Dialect/Utils/IndexingUtils.h ?
Given that I am seeing a lot of linearize / delinearize business in there I thought I would surface.
Bonus points to refactoring this enough that it could become a new helper directly in IndexingUtils.h, but this may be a bridge too far, haven't thought about this deeply enough yet.