Adding support for delegate map in emitter #334
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
facebook-github-bot
added
the
CLA Signed
|
This pull request was exported from Phabricator. Differential Revision: D49173918 |
tarun292
force-pushed
the
export-D49173918
branch
from
91eb609 to
516c291
Compare
|
This pull request was exported from Phabricator. Differential Revision: D49173918 |
tarun292
force-pushed
the
export-D49173918
branch
from
516c291 to
d7d4c77
Compare
|
This pull request was exported from Phabricator. Differential Revision: D49173918 |
tarun292
force-pushed
the
export-D49173918
branch
from
d7d4c77 to
fdcabe8
Compare
|
This pull request was exported from Phabricator. Differential Revision: D49173918 |
tarun292
force-pushed
the
export-D49173918
branch
from
fdcabe8 to
d240496
Compare
|
This pull request was exported from Phabricator. Differential Revision: D49173918 |
Summary:
LoweredBackendModules might optionally have a delegate debug handle map embedded in them if `preprocess` returned a result with a map inside it.
The emitter internally generates a mapping which is:
```
{'instruction_id' : {'name':delegate_name, 'debug_handle_map': delegate_identifier_map}
```
and returns this in the `EmitterOutput` object which is then subsequently copied over into the `ExecutorchProgram` object returned by `to_executorch()`. This enables us to serialize this map into etrecord which we can then use later for mapping back delegate internal operations to the original node in the graph.
Reviewed By: Jack-Khuu
Differential Revision: D49173918
tarun292
force-pushed
the
export-D49173918
branch
from
d240496 to
d8703df
Compare
|
This pull request was exported from Phabricator. Differential Revision: D49173918 |
|
This pull request has been merged in 1200d59. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary:
LoweredBackendModules might optionally have a delegate debug handle map embedded in them if
preprocessreturned a result with a map inside it.The emitter internally generates a mapping which is:
and returns this in the
EmitterOutputobject which is then subsequently copied over into theExecutorchProgramobject returned byto_executorch(). This enables us to serialize this map into etrecord which we can then use later for mapping back delegate internal operations to the original node in the graph.Differential Revision: D49173918