The original patch https://reviews.llvm.org/D110540 already intended for these operations to be in the arithmetic dialect.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
We've been seeing a failure in our Ubuntu builds since this change - specifically because of an unused function:
FAILED: tools/mlir/lib/Dialect/StandardOps/CMakeFiles/obj.MLIRStandard.dir/IR/Ops.cpp.o /mnt/vss/_work/1/s/llvm-project/mlir/lib/Dialect/StandardOps/IR/Ops.cpp:105:13: error: unused function 'printStandardBinaryOp' [-Werror,-Wunused-function] static void printStandardBinaryOp(Operation *op, OpAsmPrinter &p) { ^ 1 error generated.
I looked into it and it seems to me that after this change a large number of definitions in Ops.td are now no longer used (e.g. IntBinaryOp and FloatBinaryOp among others). Are they still relevant or should they go as well?