Skip to content

Commit

Permalink
Fix lint errors
Browse files Browse the repository at this point in the history
  • Loading branch information
vinayakdsci committed Sep 20, 2024
1 parent de47766 commit 757e945
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 5 deletions.
5 changes: 3 additions & 2 deletions onnxruntime/core/providers/iree/compiler/jit_compiler.cc
Original file line number Diff line number Diff line change
Expand Up @@ -194,8 +194,9 @@ common::Status CompilerInvocation::ImportSubgraph(const onnxruntime::GraphViewer
}

if (torch_mlir_onnx::failed(imp.ImportAll())) {
return ORT_MAKE_STATUS(ONNXRUNTIME, INVALID_GRAPH, "Failed to import nodes", ": ",
model_info.error_message(), ConsumeDiagnostics());
return ORT_MAKE_STATUS(ONNXRUNTIME, INVALID_GRAPH, "Failed to import nodes",
": ", model_info.error_message(),
ConsumeDiagnostics());
}

// Verify the function at the point of import because we have better diagnostics.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,8 @@
#include "mlir-c/IR.h"
#include "onnx/onnx_pb.h"

#include <optional>
#include <memory>

Check warning on line 24 in onnxruntime/core/providers/iree/compiler/torch-mlir-import-onnx/OnnxImporter.h

View workflow job for this annotation

GitHub Actions / Optional Lint C++

[cpplint] reported by reviewdog 🐶 Found C++ system header after other header. Should be: OnnxImporter.h, c system, c++ system, other. [build/include_order] [4] Raw Output: onnxruntime/core/providers/iree/compiler/torch-mlir-import-onnx/OnnxImporter.h:24: Found C++ system header after other header. Should be: OnnxImporter.h, c system, c++ system, other. [build/include_order] [4]
#include <optional>
#include <string_view>
#include <unordered_map>

Expand Down Expand Up @@ -66,7 +66,8 @@ static inline bool failed(Status status) { return !status.is_success(); }
// Accounting for a GraphProto.
class GraphInfo {
public:
GraphInfo(const onnxruntime::GraphViewer &gv, ModelInfo &model_info, const onnx::GraphProto &graph_proto)
GraphInfo(const onnxruntime::GraphViewer &gv, ModelInfo &model_info,
const onnx::GraphProto &graph_proto)
: graph_viewer_(gv), model_info_(model_info), graph_proto_(graph_proto) {}
ModelInfo &model_info() { return model_info_; }
const onnx::GraphProto &graph_proto() { return graph_proto_; }
Expand Down
2 changes: 1 addition & 1 deletion onnxruntime/python/onnxruntime_pybind_state.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1141,7 +1141,7 @@ std::unique_ptr<IExecutionProvider> CreateExecutionProviderInstance(
#endif
} else if (type == kIreeExecutionProvider) {
#if USE_IREE
const auto &it = provider_options_map.find(type);
const auto& it = provider_options_map.find(type);
ProviderOptions iree_option_map = ProviderOptions{};
if (it != provider_options_map.end()) {
iree_option_map = it->second;
Expand Down

0 comments on commit 757e945

Please sign in to comment.