onnxruntime
Clarify and improve ownership semantics Model Editor C/C++ API
#28800
Merged
Merged
Clarify and improve ownership semantics Model Editor C/C++ API #28800
yuslepukhin
merged
18 commits into
main
from
yuslepukhin/revert_model_editor_change
Revert "Prevent double-free in OrtModelEditorApi ownership transfer (…
c1c4dfac
Reject duplicate raw-pointer ownership transfer in OrtModelEditorApi
3b2504e3
Add regression tests for duplicate raw-pointer guards in OrtModelEdit…
1c8f985c
Address review feedback: validation, duplicate-name, doc clarifications
e16bfb54
Make OrtModelEditorApi take-ownership APIs strongly exception-safe
e7dc1fe4
Reword C API docs as atomicity; document and fix C++ exception safety
1a2a90cc
Use kSmallTensorExternalDataThreshold for external-initializer size c…
17d69cb3
Combine duplicate-name validation with try_emplace in AddInitializerT…
d01489c9
Clarify size-guard rationale in AddInitializerToGraph(data_is_externa…
8113a5be
Add tests for new ModelEditor C-API validation paths
63548e47
Fix build error
3c0cf048
Address review comments
28fc8f38
yuslepukhin
commented
on 2026-06-05
yuslepukhin
commented
on 2026-06-05
yuslepukhin
commented
on 2026-06-05
Address review comments
2dbf22ea
Address build failure
036dab71
Make AddInitializerToGraph small-external error message actionable
ac1943a2
Fix MSVC build: use const std::vector<int64_t>& for MakeCpuFloatTenso…
44bb34f1
tianleiwu
commented
on 2026-06-06
Address review comments
7692431b
Address review comments: fix doc, reorder checks, move node->id after…
bea8b9c4
sanaa-hamel-microsoft
approved these changes
on 2026-06-08
Assignees
No one assigned
Labels
release:1.27.0
Milestone
No milestone
Login to write a write a comment.
Login via GitHub