-
Notifications
You must be signed in to change notification settings - Fork 21.4k
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
[Vulkan][Optimize for Mobile] Avoid dereferencing element [0] if the vector is empty #92918
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/92918
Note: Links to docs will display an error until the docs builds have been completed. ❌ 6 FailuresAs of commit 608ec0b: NEW FAILURES - The following jobs have failed:
BROKEN TRUNK - The following jobs failed but were present on the merge base 438f12d:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
This pull request was exported from Phabricator. Differential Revision: D41296037 |
This pull request was exported from Phabricator. Differential Revision: D41296037 |
582e12e
to
0490550
Compare
…vector is empty (pytorch#92918) Summary: Pull Request resolved: pytorch#92918 Avoid dereferencing element [0] if the vector is empty. ___ In ```transferInputOutputBackends```, one of the rewrite passes for Vulkan ```optimize_for_mobile```, an out of bounds access happens when trying to insert a backend transfer for an input if that input's ```uses()``` is empty. This diff corrects that issue. Test Plan: Run tests ___ Phabricator + CI Tests Reviewed By: SS-JIA Differential Revision: D41296037 fbshipit-source-id: 5e8219faf020104435734c1fb25af612d8408f5a
0490550
to
cd09cb3
Compare
This pull request was exported from Phabricator. Differential Revision: D41296037 |
…vector is empty (pytorch#92918) Summary: Pull Request resolved: pytorch#92918 Avoid dereferencing element [0] if the vector is empty. ___ In ```transferInputOutputBackends```, one of the rewrite passes for Vulkan ```optimize_for_mobile```, an out of bounds access happens when trying to insert a backend transfer for an input if that input's ```uses()``` is empty. This diff corrects that issue. Test Plan: Run tests ___ Phabricator + CI Tests Reviewed By: SS-JIA Differential Revision: D41296037 fbshipit-source-id: 54b60872b1a8e4a02ddb60fa034ed75db2bf86f1
This pull request was exported from Phabricator. Differential Revision: D41296037 |
cd09cb3
to
bb0a9d3
Compare
…vector is empty (pytorch#92918) Summary: Pull Request resolved: pytorch#92918 Avoid dereferencing element [0] if the vector is empty. ___ In ```transferInputOutputBackends```, one of the rewrite passes for Vulkan ```optimize_for_mobile```, an out of bounds access happens when trying to insert a backend transfer for an input if that input's ```uses()``` is empty. This diff corrects that issue. Test Plan: Run tests ___ Phabricator + CI Tests Reviewed By: SS-JIA Differential Revision: D41296037 fbshipit-source-id: 8abe599d7e26ed76622621ff64e4a4b50b8cf1bd
This pull request was exported from Phabricator. Differential Revision: D41296037 |
bb0a9d3
to
0c9023a
Compare
…vector is empty (pytorch#92918) Summary: Pull Request resolved: pytorch#92918 Avoid dereferencing element [0] if the vector is empty. ___ In ```transferInputOutputBackends```, one of the rewrite passes for Vulkan ```optimize_for_mobile```, an out of bounds access happens when trying to insert a backend transfer for an input if that input's ```uses()``` is empty. This diff corrects that issue. Test Plan: Run tests ___ Phabricator + CI Tests Reviewed By: SS-JIA, dmm-fb Differential Revision: D41296037 fbshipit-source-id: ec82cb7ea48425a59e44f3132e2e3f6baa442a24
…vector is empty (pytorch#92918) Summary: Pull Request resolved: pytorch#92918 Avoid dereferencing element [0] if the vector is empty. ___ In ```transferInputOutputBackends```, one of the rewrite passes for Vulkan ```optimize_for_mobile```, an out of bounds access happens when trying to insert a backend transfer for an input if that input's ```uses()``` is empty. This diff corrects that issue. Test Plan: Run tests ___ Phabricator + CI Tests Reviewed By: SS-JIA, dmm-fb Differential Revision: D41296037 fbshipit-source-id: b0b385bff829f1b74b6aa4a4c72c0220fc9450bf
0c9023a
to
608ec0b
Compare
This pull request was exported from Phabricator. Differential Revision: D41296037 |
@pytorchbot merge (Initiating merge automatically since Phabricator Diff has merged) |
Merge startedYour change will be merged once all checks pass (ETA 0-4 Hours). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
Summary:
Avoid dereferencing element [0] if the vector is empty.
In
transferInputOutputBackends
, one of the rewrite passes for Vulkanoptimize_for_mobile
, an out of bounds access happens when trying to insert a backend transfer for an input if that input'suses()
is empty. This diff corrects that issue.Test Plan:
Run tests
Phabricator + CI Tests
Reviewed By: SS-JIA
Differential Revision: D41296037