Allow querying all backends registered with the executorch runtime#8521
Allow querying all backends registered with the executorch runtime#8521omerjerk wants to merge 1 commit intopytorch:mainfrom
Conversation
Summary: Partially resolves: pytorch#8267 Differential Revision: D69754074
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/8521
Note: Links to docs will display an error until the docs builds have been completed. ❌ 1 New FailureAs of commit 53b5289 with merge base 5e4d6b6 ( NEW FAILURE - The following job has failed:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
|
This pull request was exported from Phabricator. Differential Revision: D69754074 |
This PR needs a
|
|
Adding reviewers in the top-right corner of this PR page will help get your PR looked at. std::vector cannot be used in runtime/backend; it is part of ExecuTorch core. See https://github.com/pytorch/executorch/blob/main/CONTRIBUTING.md#cc-standard-library-usage . (We should update that document to clarify what is in ExecuTorch core and what is not...) |
swolchok
left a comment
There was a problem hiding this comment.
for std::vector. see top-level comment on PR. not sure offhand how to get this done without heap allocation but I imagine there are other examples of "enumeration functions" as mentioned in the issue. I would recommend just checking with issue owner @GregoryComer
|
Looks like this PR hasn't been updated in a while so we're going to go ahead and mark this as |
|
Looks like this PR hasn't been updated in a while so we're going to go ahead and mark this as |
1 similar comment
|
Looks like this PR hasn't been updated in a while so we're going to go ahead and mark this as |
Summary: Partially resolves: #8267
Differential Revision: D69754074