Tong Chen

Optional input of Identity Op in Opset 16

2

# Ask a Question Why the "optional" is marked with each allowed data type, instead of with the input? ### Question The type constraint `V` for input has a list...

Documentation update for onnx version

1

Signed-off-by: chentong319 Add some instructions.

Record all the backend test cases

4

This PR records all the backend test cases, models and nodes, in file all_test_names.txt. User can use this file to add new backend test cases without going into third_party/onnx code....

issues with mlir2FileCheck.py

1

This is a very useful tool for lit test. But there are several issues with the tool: 1. Arguments in block. They should be definition of a symbol, not a...

Test_loop13_seq.onnx can not be compiled. The main reason is !onnx.Seq. I just record my investigation with this issue. I do not think that we will work on this issue in...

Handle onnx.loop that behaves like a while loop

1

The current implementation of conversion from onnx.loop to krnl assumes that the onnx.loop is an affine loop. For example, the onnx.loop in test_loop11_cpu.onnx is lowered to the following code, in...

Problem caused by the "name" attribute added to ONNXConstantOp

1

The problem is with check-onnx-lit. In the onnx-to-krnl pass, a name attribute is added to ConstantOp with unique ID, such as "constant_0" and "constant_1". When a test case is added...

Documentation related issues, questions and possible work items

6

This issue is opened to record issues, questions and possible work items when we update onnx-mlir documents. 1. Are we following docs/Documentation.md now? What's the role of docs/_data/navigation.yml? 2. What's...

Different version of Slice Op

9

@amanwithfruit I am debugging a model and found the error was in ImportNodeSlice. The size for DenseElementAttribute is assumed to be `{1}`, while the attribute in the test case has...

I noticed that the instrumentation tool for onnx duplicated to support ZHigh dialect. Other than code duplication, the current tool cannot easily support IR with mixed dialect. I think the...