Conv3D has an existing linalg operation for floating point. Adding a quantized
variant and corresponding lowering from TOSA. Numerical correctness was validated
using the TOSA conformance tests.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
mlir/lib/Conversion/TosaToLinalg/TosaToLinalgNamed.cpp | ||
---|---|---|
112–127 | Here and below please follow https://llvm.org/docs/CodingStandards.html#don-t-evaluate-end-every-time-through-a-loop (also prefer to pre-increment except if you need post-increment behavior) | |
121 | Is getResult needed? I think there is implicit conversion if single result op. | |
364–365 | Could you add a comment for these? | |
mlir/python/mlir/dialects/linalg/opdsl/ops/core_named_ops.py | ||
481 | The zero points offset, is that a restriction for when lowering this op that needs to be enforced? (I don't see anything here checking for that, so I assume its in the lowering). | |
mlir/test/Conversion/TosaToLinalg/tosa-to-linalg-named.mlir | ||
658 | please add new line |
mlir/lib/Conversion/TosaToLinalg/TosaToLinalgNamed.cpp | ||
---|---|---|
112–127 | I am a bit confused, I thought I had already done the evaluation preference using s? Updated for pre-increment. | |
mlir/python/mlir/dialects/linalg/opdsl/ops/core_named_ops.py | ||
481 | The zero-point offsets are the addition IZp and KZp arguments to the operation. |
Is getResult needed? I think there is implicit conversion if single result op.