Skip to content

Try adding serveral Operators to the IR #678

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Try adding serveral Operators to the IR #678

wants to merge 5 commits into from

Conversation

YanjieGao
Copy link

@YanjieGao YanjieGao commented Jun 11, 2019

Try adding several OPs (Follow ONNX) to the IR. Please verify the standard.

Plan:
1 keep the legacy IR file ops.pbtxt
2 add new op (Not in legacy ops.pbtxt) at ops_extend.pbtxt which not in ops.pbtxt. This phase will merge onnx operators.

Exclude op below:
legacy_operator_set = ["Add", "AveragePool", "BatchNormalization", "Concat", "Conv", "ConvTranspose", "Div", "Dropout", "Elu",
"Exp", "Flatten", "GRU", "Gather", "Gemm", "GlobalAveragePool", "GlobalMaxPool", "LRN", "LSTM", "LeakyRelu", "Log", "MatMul",
"Mul", "Pad", "RNN", "Reciprocal", "Relu", "Reshape", "Sigmoid", "Slice", "Softmax", "Split", "Sequeeze", "Sub", "Tanh"]

3 Future new IR op will add to ops_extend.pbtxt
4 When the legacy IR plan want to update, it needs to be updated with the pbtxt file, parser and emit, and then updated.

@linmajia linmajia requested a review from rainLiuplus June 17, 2019 05:43
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant