add GatherSliceToSplitFusion and Unittest #25533
Annotations
9 warnings
Run reviewdog/action-cpplint@master:
onnxruntime/core/optimizer/gather_slice_fusion.cc#L298
[cpplint] reported by reviewdog 🐶
If an else has a brace on one side, it should have it on both [readability/braces] [5]
Raw Output:
onnxruntime/core/optimizer/gather_slice_fusion.cc:298: If an else has a brace on one side, it should have it on both [readability/braces] [5]
|
Run reviewdog/action-cpplint@master:
onnxruntime/core/optimizer/gather_slice_fusion.cc#L310
[cpplint] reported by reviewdog 🐶
Closing ) should be moved to the previous line [whitespace/parens] [2]
Raw Output:
onnxruntime/core/optimizer/gather_slice_fusion.cc:310: Closing ) should be moved to the previous line [whitespace/parens] [2]
|
Run reviewdog/action-cpplint@master:
onnxruntime/core/optimizer/gather_slice_fusion.cc#L351
[cpplint] reported by reviewdog 🐶
Namespace should be terminated with "// namespace onnxruntime" [readability/namespace] [5]
Raw Output:
onnxruntime/core/optimizer/gather_slice_fusion.cc:351: Namespace should be terminated with "// namespace onnxruntime" [readability/namespace] [5]
|
Run reviewdog/action-cpplint@master:
onnxruntime/core/optimizer/gather_slice_fusion.h#L17
[cpplint] reported by reviewdog 🐶
private: should be indented +1 space inside class GatherSliceToSplitFusion [whitespace/indent] [3]
Raw Output:
onnxruntime/core/optimizer/gather_slice_fusion.h:17: private: should be indented +1 space inside class GatherSliceToSplitFusion [whitespace/indent] [3]
|
Run reviewdog/action-cpplint@master:
onnxruntime/core/optimizer/gather_slice_fusion.h#L27
[cpplint] reported by reviewdog 🐶
public: should be indented +1 space inside class GatherSliceToSplitFusion [whitespace/indent] [3]
Raw Output:
onnxruntime/core/optimizer/gather_slice_fusion.h:27: public: should be indented +1 space inside class GatherSliceToSplitFusion [whitespace/indent] [3]
|
Run reviewdog/action-cpplint@master:
onnxruntime/core/optimizer/gather_slice_fusion.h#L28
[cpplint] reported by reviewdog 🐶
Constructors callable with one argument should be marked explicit. [runtime/explicit] [5]
Raw Output:
onnxruntime/core/optimizer/gather_slice_fusion.h:28: Constructors callable with one argument should be marked explicit. [runtime/explicit] [5]
|
Run reviewdog/action-cpplint@master:
onnxruntime/core/optimizer/gather_slice_fusion.h#L32
[cpplint] reported by reviewdog 🐶
Redundant blank line at the end of a code block should be deleted. [whitespace/blank_line] [3]
Raw Output:
onnxruntime/core/optimizer/gather_slice_fusion.h:32: Redundant blank line at the end of a code block should be deleted. [whitespace/blank_line] [3]
|
Run reviewdog/action-cpplint@master:
onnxruntime/core/optimizer/gather_slice_fusion.h#L34
[cpplint] reported by reviewdog 🐶
Namespace should be terminated with "// namespace onnxruntime" [readability/namespace] [5]
Raw Output:
onnxruntime/core/optimizer/gather_slice_fusion.h:34: Namespace should be terminated with "// namespace onnxruntime" [readability/namespace] [5]
|
Run reviewdog/action-cpplint@master:
onnxruntime/test/optimizer/graph_transform_test.cc#L7725
[cpplint] reported by reviewdog 🐶
You don't need a ; after a } [readability/braces] [4]
Raw Output:
onnxruntime/test/optimizer/graph_transform_test.cc:7725: You don't need a ; after a } [readability/braces] [4]
|
The logs for this run have expired and are no longer available.
Loading