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.