-
Notifications
You must be signed in to change notification settings - Fork 696
Arm backend: Move rescales from ADD & SUB visitors to pass #15378
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?
Conversation
Move the insertion of INT8/INT32 RESCALE ops from the ADD node visitor to the pass InsertRescaleInt32Pass. This is in practice a refactoring patch, but still the output TOSA file becomes different enough to cause an Ethos-U55 test to fail in test_conv_relu_residual_add.py. However, this issue was fixed in https://gitlab.arm.com/artificial-intelligence/ethos-u/ethos-u-vela/-/commit/642f7517d3a6bd053032e1942822f6e38ccd546f so we temporarily set the failing test to xfail until the version of Ethos-U Vela compiler depended on is bumped to one that includes the fix. Signed-off-by: Martin Lindström <Martin.Lindstroem@arm.com> Co-authored-by: Oscar Andersson <Oscar.Andersson@arm.com> Change-Id: I90cd9ab5a296911a228b0080008e4c65ba773f7c
Move the insertion of INT8/INT32 RESCALE ops from the SUB node visitor to the pass InsertRescaleInt32Pass. This is in practice a refactoring patch, but still the output TOSA file becomes different enough to cause an Ethos-U55 test to fail in test_var.py. However, this issue was fixed in https://gitlab.arm.com/artificial-intelligence/ethos-u/ethos-u-vela/-/commit/642f7517d3a6bd053032e1942822f6e38ccd546f so we temporarily set the failing test to xfail until the version of Ethos-U Vela compiler depended on is bumped to one that includes the fix. Signed-off-by: Martin Lindstroem <Martin.Lindstroem@arm.com> Co-authored-by: Oscar Andersson <Oscar.Andersson@arm.com> Change-Id: I38d63015e03e59c267338c84d64731b050854d06
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/15378
Note: Links to docs will display an error until the docs builds have been completed. ❌ 8 New Failures, 1 Cancelled Job, 3 Unrelated FailuresAs of commit 07d8738 with merge base de56c81 ( NEW FAILURES - The following jobs have failed:
CANCELLED JOB - The following job was cancelled. Please retry:
FLAKY - The following job failed but was likely due to flakiness present on trunk:
BROKEN TRUNK - The following jobs failed but was present on the merge base:👉 Rebase onto the `viable/strict` branch to avoid these failures
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
|
@pytorchbot label ciflow/trunk |
|
@pytorchbot label "partner: arm" |
This PR needs a
|
Move the insertion of INT8/INT32 RESCALE ops from the SUB node visitor to the pass InsertRescaleInt32Pass.
This is in practice a refactoring patch, but still the output TOSA files becomes different enough to cause Ethos-U55/U85 tests to fail in test_var.py and test_conv_relu_residual_add.py. However, the issue was fixed in https://gitlab.arm.com/artificial-intelligence/ethos-u/ethos-u-vela/-/commit/642f7517d3a6bd053032e1942822f6e38ccd546f so we temporarily set the failing tests to xfail until the version of Ethos-U Vela compiler depended on is bumped to one that includes the fix.
Test plan
test_insert_rescale_i32_pass.py has been modified to test the change.
cc @freddan80 @per @zingo @oscarandersson8218 @digantdesai