fix:lint fixes

Author: ankitm3k

onnxruntime/core/providers/openvino/backend_manager.cc

@@ -44,6 +44,10 @@ BackendManager::BackendManager(SessionContext& session_context,
                                                               shared_context_{shared_context} {
   subgraph_context_.is_ep_ctx_graph = ep_ctx_handle_.CheckForOVEPCtxNodeInGraph(subgraph);
 
+  bool cpu_or_gpu = session_context_.device_type.find("CPU") != std::string::npos ||
+                    session_context_.device_type.find("GPU") != std::string::npos;
+  bool npu = session_context_.device_type.find("NPU") != std::string::npos;
+
   subgraph_context_.model_precision = [&](const GraphViewer& graph_viewer) {
     // return empty if graph has no inputs or if types are not one of FP32/FP16
     // else assume the type of the first input
@@ -105,9 +109,7 @@ BackendManager::BackendManager(SessionContext& session_context,
   if (ModelHasSymbolicInputDims(subgraph)) {
     subgraph_context_.has_dynamic_input_shape = true;
     LOGS_DEFAULT(INFO) << "[OpenVINO-EP] Model has symbolic input dims";
-    if ((session_context_.device_type.find("CPU") != std::string::npos ||
-         session_context_.device_type.find("GPU") != std::string::npos ||
-         (session_context_.device_type.find("NPU") != std::string::npos && session_context_.enable_causallm)) &&
+    if (cpu_or_gpu || (npu && session_context_.enable_causallm) &&
         !session_context_.disable_dynamic_shapes) {
       LOGS_DEFAULT(INFO) << "[OpenVINO-EP] Starting backend initialization. "
                          << "Creating backend Dynamic Shapes";
@@ -481,6 +483,9 @@ BackendManager::ReWriteBatchDimWithOne(const ONNX_NAMESPACE::ModelProto& model_p
 void BackendManager::Compute(OrtKernelContext* context) {
   Ort::KernelContext ctx(context);
   std::chrono::high_resolution_clock::time_point start_compute, end_compute;
+  bool cpu_or_gpu = session_context_.device_type.find("CPU") != std::string::npos ||
+                    session_context_.device_type.find("GPU") != std::string::npos;
+  bool npu = session_context_.device_type.find("NPU") != std::string::npos;
 #ifdef OPENVINO_FIL_ENABLED
   static bool fil_enabled = true;
   if (fil_enabled) {
@@ -494,10 +499,7 @@ void BackendManager::Compute(OrtKernelContext* context) {
   // disable_dynamic_shapes is always set to true for OV NPU plugin.
   if (subgraph_context_.has_dynamic_input_shape &&
       !session_context_.disable_dynamic_shapes &&
-      (session_context_.device_type.find("CPU") != std::string::npos ||
-       session_context_.device_type.find("GPU") != std::string::npos ||
-       (session_context_.device_type.find("NPU") != std::string::npos &&
-        session_context_.enable_causallm))) {
+      (cpu_or_gpu || (npu && session_context_.enable_causallm))) {
     concrete_backend_->Infer(context);
   } else if (subgraph_context_.has_dynamic_input_shape) {
     std::vector<std::vector<int64_t>> tensor_shapes = GetInputTensorShapes(ctx);

onnxruntime/core/providers/openvino/ibackend.h

@@ -17,7 +17,7 @@ class IBackend {
   virtual void Infer(OrtKernelContext* context) = 0;
   virtual ov::CompiledModel GetOVCompiledModel() = 0;
   virtual ~IBackend() = default;
-  virtual void RewindKVCache(size_t index) {};
+  virtual void RewindKVCache(size_t index) {}
 };
 using ptr_stream_t = std::unique_ptr<std::istream>;
 class BackendFactory {

onnxruntime/core/providers/openvino/ov_interface.cc

@@ -392,7 +392,7 @@ StatefulOVInferRequest::StatefulOVInferRequest(ov::InferRequest infer_request, s
 
 void StatefulOVInferRequest::PreProcessInferRequest() {
   // Workaround: Setting the value here as it cannot be set at the ORT GenAI layer currently.
-  // TODO: Address this issue and implement the fix at the appropriate layer.
+  // TODO(ankit): Address this issue and implement the fix at the appropriate layer.
   ov::Tensor beam_idx = ov::Tensor(ov::element::i32, {1});
   std::fill_n(beam_idx.data<int32_t>(), 1, 0);
   ovInfReq.set_tensor("beam_idx", beam_idx);

onnxruntime/core/providers/openvino/ov_interface.h

@@ -136,7 +136,7 @@ class OVInferRequest {
   ov::InferRequest& GetNewObj() {
     return ovInfReq;
   }
-  virtual void RewindKVCache(size_t index) {};
+  virtual void RewindKVCache(size_t index) {}
 };
 
 class StatefulOVInferRequest : public OVInferRequest {

onnxruntime/core/providers/openvino/ov_stateful_patch_utils.cc

@@ -1,7 +1,7 @@
 // Copyright (C) Intel Corporation
 // Licensed under the MIT License
 
-#include "ov_stateful_patch_utils.h"
+#include "core/providers/openvino/ov_stateful_patch_utils.h"
 
 namespace onnxruntime {
 namespace openvino_ep {
@@ -155,7 +155,7 @@ void PatchStatefulDecoder(std::shared_ptr<ov::Model> model) {
   }
 
   // By default, batch is the 0 - th but chatglm uses 1 - st dimension as batch
-  // TODO: Deduce from a model via ordinal reshape(? ) and topology
+  // TODO(ryan): Deduce from a model via ordinal reshape(? ) and topology
   // batch_dim = 1 if config.model_type == "chatglm" and not hasattr(config, "rope_ratio") else 0
   auto batch_dim = 0;