Skip to content

Cherry-pick #25830 for 1.23.0 release - #25921

Closed
nieubank wants to merge 1 commit into
rel-1.23.0from
user/nieubank/rel-1.23_25879
Closed

Cherry-pick #25830 for 1.23.0 release#25921
nieubank wants to merge 1 commit into
rel-1.23.0from
user/nieubank/rel-1.23_25879

Conversation

@nieubank

@nieubank nieubank commented Sep 2, 2025

Copy link
Copy Markdown
Contributor

## Summary
Adds EP metadata library path support to enable custom ops DLL
registration with proper path resolution.

## Changes
- Added `library_path` metadata key to EP metadata infrastructure
- Pass resolved library path directly to `EpLibraryProviderBridge`
constructor
- Simplified implementation per reviewer feedback (removed virtual
method complexity)
- Added `#include <utility>` for std::move compliance

## Purpose
Enables downstream applications (like onnxruntime-genai) to resolve
relative custom ops library paths using EP metadata, improving DLL
registration reliability.

## Files Modified
- `plugin_ep/ep_factory_provider_bridge.h`
- `plugin_ep/ep_library.h` 
- `plugin_ep/ep_library_plugin.h`
- `plugin_ep/ep_library_provider_bridge.cc`
- `plugin_ep/ep_library_provider_bridge.h`
- `utils.cc`
@nieubank nieubank changed the title Cherry-picks #25830 for 1.23.0 release Cherry-pick #25830 for 1.23.0 release Sep 2, 2025
@jywu-mysoft

Copy link
Copy Markdown
Contributor

we normally cherry pick in bulk
@snnn should be creating another cherry pick PR soon

@nieubank

nieubank commented Sep 2, 2025

Copy link
Copy Markdown
Contributor Author

we normally cherry pick in bulk @snnn should be creating another cherry pick PR soon

That's what I thought, I just noticed that the closed PR still has the 1.23 tag, so I'll close this one and make sure I follow the coming CP PRs.

@nieubank nieubank closed this Sep 2, 2025
@snnn
snnn deleted the user/nieubank/rel-1.23_25879 branch September 5, 2025 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants