Add entry_at(i) accessor to LoadBackendOptionsMap (#19645)#19645
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/19645
Note: Links to docs will display an error until the docs builds have been completed. ❗ 1 Active SEVsThere are 1 currently active SEVs. If your PR is affected, please view them below: ✅ You can merge normally! (1 Unrelated Failure)As of commit e9b5d80 with merge base 30c9a26 ( FLAKY - The following job failed but was likely due to flakiness present on trunk:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
|
@metascroy has exported this pull request. If you are a Meta employee, you can view the originating Diff in D105100370. |
This PR needs a
|
Summary:
LoadBackendOptionsMap exposed get_options(backend_id), has_options(backend_id),
and size() but no way to enumerate the entries. That makes deep-copy,
debugging/pretty-printing, and any other operation that wants to iterate
over the configured backends impossible from outside the class.
Add a public EntryView struct ({backend_id, options}) plus an entry_at(i)
accessor returning a non-owning view at the given index. Header-only,
additive; existing callers are unaffected.
Reviewed By: shoumikhin
Differential Revision: D105100370
cbec41c to
e9b5d80
Compare
Summary:
LoadBackendOptionsMap exposed get_options(backend_id), has_options(backend_id),
and size() but no way to enumerate the entries. That makes deep-copy,
debugging/pretty-printing, and any other operation that wants to iterate
over the configured backends impossible from outside the class.
Add a public EntryView struct ({backend_id, options}) plus an entry_at(i)
accessor returning a non-owning view at the given index. Header-only,
additive; existing callers are unaffected.
Reviewed By: shoumikhin
Differential Revision: D105100370