Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
| mlir/lib/Dialect/Math/IR/MathOps.cpp | ||
|---|---|---|
| 109–111 | These are all guaranteed by the verifier and by the op's design. We shouldn't be checking for invalid IR. | |
| mlir/lib/Dialect/Math/IR/MathOps.cpp | ||
|---|---|---|
| 109–111 | Thanks, I create a new revision to remove these https://reviews.llvm.org/D121986. | |
| mlir/lib/Dialect/Math/IR/MathOps.cpp | ||
|---|---|---|
| 109–111 | Nope, these are not guaranteed: We just don't fold tensor or vector constants that aggressively, but it could be DenseElementsAttr. | |
| mlir/lib/Dialect/Math/IR/MathOps.cpp | ||
|---|---|---|
| 109–111 | Sorry, I was mistaken here. This is fine as is. | |
These are all guaranteed by the verifier and by the op's design. We shouldn't be checking for invalid IR.