Skip to content

Commit 864dece

Browse files
Jerry-GeTai78641
authored andcommitted
Backporting of (see below) into feature/fused-ops
[mlir][tosa] Remove FullyConnectedOp from TOSA Dialect (llvm#126152) This patch removes FullyConncected Operator from the TOSA Dialect and all associated tests and transforms. This is part of the TOSA v1.0 alignment effort: https://discourse.llvm.org/t/rfc-tosa-dialect-increment-to-v1-0/83708 Signed-off-by: Tai Ly <[email protected]> Co-authored-by: Tai Ly <[email protected]>
1 parent 7223b1f commit 864dece

File tree

17 files changed

+105
-501
lines changed

17 files changed

+105
-501
lines changed

mlir/include/mlir/Dialect/Tosa/IR/TosaOpBase.td

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -150,15 +150,6 @@ def Tosa_TransConvOpQuantInfoBuilder : OpBuilder<
150150
outputShape, acc_type);
151151
}]>;
152152

153-
// The tosa.fully_connected op has its own builder as it does not have
154-
// strides/dilation/padding.
155-
def Tosa_FCOpQuantInfoBuilder : OpBuilder<
156-
(ins "Type":$outputType, "Value":$input, "Value":$weight, "Value":$bias),
157-
[{
158-
buildFCOpWithQuantInfo($_builder, $_state, outputType,
159-
input, weight, bias);
160-
}]>;
161-
162153
// The tosa.matmul op is also intended to be generated where a fully_connected
163154
// op must be constructed where the weight is not a constant. In this case,
164155
// the fully_connected op must be expressed using matmul.

0 commit comments

Comments
 (0)