From af679a025a59d13dd0131642c0e19ebcd2685671 Mon Sep 17 00:00:00 2001 From: Javier Martinez Date: Mon, 10 Feb 2025 14:54:53 -0800 Subject: [PATCH] Fix logic for selecting alternate name for blob (#23617) ### Description When context embed mode 0 there were some unhandled corner cases in OVEP that generated inconsistent/incorrect compiled blob names. This PR corrects that. ### Motivation and Context Fix corner cases when OVEP generates external compiled blob names. --- onnxruntime/core/providers/openvino/backend_manager.cc | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/onnxruntime/core/providers/openvino/backend_manager.cc b/onnxruntime/core/providers/openvino/backend_manager.cc index 16a92b43adaf6..2a842b8a1eca8 100644 --- a/onnxruntime/core/providers/openvino/backend_manager.cc +++ b/onnxruntime/core/providers/openvino/backend_manager.cc @@ -225,13 +225,11 @@ Status BackendManager::ExportCompiledBlobAsEPCtxNode(const onnxruntime::GraphVie // Build name by combining EpCtx model name (if available) and subgraph name. Model // name is not available in when creating a session from memory auto name = session_context_.so_context_file_path.stem().string(); - if (!name.empty() && !graph_body_viewer.ModelPath().empty()) { + if (name.empty() && !graph_body_viewer.ModelPath().empty()) { name = graph_body_viewer.ModelPath().stem().string(); } - if (!name.empty()) { - name += "_"; - } - name += subgraph_context_.subgraph_name; + ORT_ENFORCE(!name.empty()); + name += "_" + subgraph_context_.subgraph_name; std::filesystem::path blob_filename = session_context_.so_context_file_path; if (blob_filename.empty()) {