This is a basic op needed for creating shapes from SSA values
representing the extents.
Details
- Reviewers
jpienaar - Commits
- rG21b0eff7738a: [mlir][shape] Add `shape.from_extents`.
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Thanks
mlir/include/mlir/Dialect/Shape/IR/ShapeOps.td | ||
---|---|---|
113 | No period at the end please (summary is sentence fragment) | |
mlir/lib/Dialect/Shape/IR/Shape.cpp | ||
193 | Seems like we could just have constantShapeReturn function which all the inferReturnType could call [I did a version where it was "trait" that reduced a lot of repetition which I should perhaps clean up a bit - doesn't add anything truly new, just packages] |
Address comments.
mlir/lib/Dialect/Shape/IR/Shape.cpp | ||
---|---|---|
193 | I want to avoid that because I specifically want to make this ugly to show the hole in the infra (and eventually make myself fix it :) ). If all result types are buildable types then this should be autogenerated somehow. (as a stretch goal unifying with the type inference logic used by the declarative asm parser) |
No period at the end please (summary is sentence fragment)