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)