Skip to content

Arm backend: Add TOSA binary op visitors#20479

Merged
SaoirseARM merged 5 commits into
pytorch:mainfrom
SaoirseARM:toupstream/bin_node
Jun 26, 2026
Merged

Arm backend: Add TOSA binary op visitors#20479
SaoirseARM merged 5 commits into
pytorch:mainfrom
SaoirseARM:toupstream/bin_node

Conversation

@SaoirseARM

@SaoirseARM SaoirseARM commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Running the cast pass first normalizes constant buffers while the graph is still Edge ATen, so ExirToTosaPass only sees int32 buffer metadata and emits TOSA-compatible ops.
  • Small refactor of ExirToTosaPass

cc @digantdesai @freddan80 @per @zingo @oscarandersson8218 @mansnils @Sebastian-Larsson @robell @rascani

- Run ExirToTosaPass after int64 cleanup.
- Small refactor of ExirToTosaPass

Signed-off-by: Saoirse Stewart <saoirse.stewart@arm.com>
@pytorch-bot

pytorch-bot Bot commented Jun 24, 2026

Copy link
Copy Markdown

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/20479

Note: Links to docs will display an error until the docs builds have been completed.

❌ 2 New Failures, 3 Unrelated Failures

As of commit f402ea6 with merge base 6021a58 (image):

NEW FAILURES - The following jobs have failed:

BROKEN TRUNK - The following jobs failed but were 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.

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jun 24, 2026
@github-actions github-actions Bot added ciflow/trunk module: arm Issues related to arm backend labels Jun 24, 2026
@SaoirseARM

Copy link
Copy Markdown
Collaborator Author

@pytorchbot label ciflow/trunk

@SaoirseARM

Copy link
Copy Markdown
Collaborator Author

@pytorchbot label 'partner: arm'

@pytorch-bot pytorch-bot Bot added the partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm label Jun 24, 2026
@SaoirseARM

Copy link
Copy Markdown
Collaborator Author

@pytorchbot label "release notes: arm"

@pytorch-bot pytorch-bot Bot added the release notes: arm Changes to the ARM backend delegate label Jun 24, 2026

@zingo zingo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK to merge if tests are OK, just make sure buck2 stuff don't need to be updated. You can just ask codex/claude

@zingo

zingo commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator

FAILED backends/arm/test/misc/test_vgf_backend.py::test_vgf_compile_failure_includes_repro_command_and_copies_tosa

Is known and a fix has been merged, just rebase or double check to there are no others files.

Erik-Lundell and others added 3 commits June 24, 2026 13:38
Update bool promotion to leave bitwise ops available for logical
rewrites, and handle tied-parameter cleanup in FuseConstantArgsPass.

Signed-off-by: Saoirse Stewart <saoirse.stewart@arm.com>
@SaoirseARM

Copy link
Copy Markdown
Collaborator Author

No buck2 changes required and failures look unrelated

@SaoirseARM SaoirseARM merged commit a2438e4 into pytorch:main Jun 26, 2026
481 of 489 checks passed
@SaoirseARM SaoirseARM deleted the toupstream/bin_node branch June 26, 2026 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunk CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. module: arm Issues related to arm backend partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm release notes: arm Changes to the ARM backend delegate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants