-
Notifications
You must be signed in to change notification settings - Fork 626
Cmsis #12655
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Cmsis #12655
Conversation
Test Plan: Reviewers: Subscribers: Tasks: Tags:
This PR needs a
|
"dtype %" PRId8 " is not int8 (Char)", | ||
static_cast<int8_t>(dtype)); | ||
|
||
assert(false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this will be a no-op for a RELEASE CMake build.
@@ -31,6 +31,10 @@ def _is_qualified_int8_node(args) -> bool: | |||
def __init__(self): | |||
super().__init__() | |||
self.op_replacements = { | |||
exir_ops.edge.aten.add.Tensor: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you print your graph before and after this pass to see if this replacement is actually happening?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes , for the softmax usecase, I see the following
[ReplaceQuantNodesPass] Operator called: <EdgeOpOverload: aten._softmax.default>: schema = aten::_softmax(Tensor self, int dim, bool half_to_float) -> Tensor, Args: (<executorch.exir.pass_base.ProxyValue object at 0x7fa982bc03a0>, 0, False)
[ReplaceQuantNodesPass] Replacing <EdgeOpOverload: aten._softmax.default>: schema = aten::_softmax(Tensor self, int dim, bool half_to_float) -> Tensor with cortex_m.softmax
@@ -27,7 +27,7 @@ namespace ET_RUNTIME_NAMESPACE { | |||
* Interface to access and retrieve data via name. | |||
* See executorch/extension/flat_tensor/ for an example. | |||
*/ | |||
class ET_EXPERIMENTAL NamedDataMap { | |||
class NamedDataMap { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
accidental?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will remove it.
38a5ce1
to
8e16851
Compare
Summary: Test Plan: Reviewers: Subscribers: Tasks: Tags:
Summary: Test Plan: Reviewers: Subscribers: Tasks: Tags:
Test Plan: Reviewers: Subscribers: Tasks: Tags:
Summary
[PLEASE REMOVE] See CONTRIBUTING.md's Pull Requests for ExecuTorch PR guidelines.
[PLEASE REMOVE] If this PR closes an issue, please add a
Fixes #<issue-id>
line.[PLEASE REMOVE] If this PR introduces a fix or feature that should be the upcoming release notes, please add a "Release notes: " label. For a list of available release notes labels, check out CONTRIBUTING.md's Pull Requests.
Test plan
[PLEASE REMOVE] How did you test this PR? Please write down any manual commands you used and note down tests that you have written if applicable.