Skip to content

[iOS GPU] Add the reset of binary ops#53950

Closed
xta0 wants to merge 1 commit intogh/xta0/93/basefrom
gh/xta0/93/head
Closed

[iOS GPU] Add the reset of binary ops#53950
xta0 wants to merge 1 commit intogh/xta0/93/basefrom
gh/xta0/93/head

Conversation

@xta0
Copy link
Copy Markdown
Contributor

@xta0 xta0 commented Mar 13, 2021

Stack from ghstack:

Add four binary ops to Metal

  • aten::mul_
  • aten::sub_
  • aten::div
  • aten::div_

Differential Revision: D27003417

Add four binary ops to Metal

- `aten::mul_`
- `aten::sub_`
- `aten::div`
- `aten::div_`

Differential Revision: [D27003417](https://our.internmc.facebook.com/intern/diff/D27003417/)

[ghstack-poisoned]
@facebook-github-bot
Copy link
Copy Markdown
Contributor

facebook-github-bot commented Mar 13, 2021

💊 CI failures summary and remediations

As of commit eedacb8 (more details on the Dr. CI page):


  • 1/1 failures possibly* introduced in this PR
    • 1/1 non-scanned failure(s)

This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions to the (internal) Dr. CI Users group.

xta0 added a commit that referenced this pull request Mar 13, 2021
Add four binary ops to Metal

- `aten::mul_`
- `aten::sub_`
- `aten::div`
- `aten::div_`

Differential Revision: [D27003417](https://our.internmc.facebook.com/intern/diff/D27003417/)

ghstack-source-id: 123850577
Pull Request resolved: #53950
@xta0 xta0 requested a review from SS-JIA March 13, 2021 03:37
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 13, 2021

Codecov Report

Merging #53950 (eedacb8) into gh/xta0/93/base (1d762cf) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@                 Coverage Diff                 @@
##           gh/xta0/93/base   #53950      +/-   ##
===================================================
- Coverage            77.30%   77.30%   -0.01%     
===================================================
  Files                 1888     1888              
  Lines               183589   183589              
===================================================
- Hits                141921   141917       -4     
- Misses               41668    41672       +4     

@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request has been merged in bb21aea.

@facebook-github-bot facebook-github-bot deleted the gh/xta0/93/head branch March 18, 2021 14:16
xsacha pushed a commit to xsacha/pytorch that referenced this pull request Mar 31, 2021
Summary:
Pull Request resolved: pytorch#53950

Add four binary ops to Metal

- `aten::mul_`
- `aten::sub_`
- `aten::div`
- `aten::div_`
ghstack-source-id: 123850577

Test Plan:
- `buck test pp-mac`

```
2021-03-11 20:36:47.151139-0800 PyTorchPlayground[8469:5169786] [bool test_sub()],[5 3 167 222 ],[SUCCEED]
2021-03-11 20:36:47.157638-0800 PyTorchPlayground[8469:5169786] [bool test_sub_broadcast()],[1 3 1 1 ],[SUCCEED]
2021-03-11 20:36:47.170640-0800 PyTorchPlayground[8469:5169786] [bool test_sub_broadcast2()],[3 3 192 192 ],[SUCCEED]
2021-03-11 20:36:47.194009-0800 PyTorchPlayground[8469:5169786] [bool test_mul()],[2 7 262 119 ],[SUCCEED]
2021-03-11 20:36:47.210344-0800 PyTorchPlayground[8469:5169786] [bool test_mul_broadcast()],[4 3 192 192 ],[SUCCEED]
2021-03-11 20:36:47.216610-0800 PyTorchPlayground[8469:5169786] [bool test_mul_broadcast2()],[1 3 192 192 ],[SUCCEED]
2021-03-11 20:36:47.224471-0800 PyTorchPlayground[8469:5169786] [bool test_div()],[1 3 192 192 ],[SUCCEED]
2021-03-11 20:36:47.240817-0800 PyTorchPlayground[8469:5169786] [bool test_div_broadcast()],[4 3 192 192 ],[SUCCEED]
2021-03-11 20:36:47.246816-0800 PyTorchPlayground[8469:5169786] [bool test_div_broadcast2()],[1 3 192 192 ],[SUCCEED]
```

Reviewed By: SS-JIA

Differential Revision: D27003417

fbshipit-source-id: 290f7e524eef4c444f8884fc1315151752e5ac31
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
Pull Request resolved: pytorch#53950

Add four binary ops to Metal

- `aten::mul_`
- `aten::sub_`
- `aten::div`
- `aten::div_`
ghstack-source-id: 123850577

Test Plan:
- `buck test pp-mac`

```
2021-03-11 20:36:47.151139-0800 PyTorchPlayground[8469:5169786] [bool test_sub()],[5 3 167 222 ],[SUCCEED]
2021-03-11 20:36:47.157638-0800 PyTorchPlayground[8469:5169786] [bool test_sub_broadcast()],[1 3 1 1 ],[SUCCEED]
2021-03-11 20:36:47.170640-0800 PyTorchPlayground[8469:5169786] [bool test_sub_broadcast2()],[3 3 192 192 ],[SUCCEED]
2021-03-11 20:36:47.194009-0800 PyTorchPlayground[8469:5169786] [bool test_mul()],[2 7 262 119 ],[SUCCEED]
2021-03-11 20:36:47.210344-0800 PyTorchPlayground[8469:5169786] [bool test_mul_broadcast()],[4 3 192 192 ],[SUCCEED]
2021-03-11 20:36:47.216610-0800 PyTorchPlayground[8469:5169786] [bool test_mul_broadcast2()],[1 3 192 192 ],[SUCCEED]
2021-03-11 20:36:47.224471-0800 PyTorchPlayground[8469:5169786] [bool test_div()],[1 3 192 192 ],[SUCCEED]
2021-03-11 20:36:47.240817-0800 PyTorchPlayground[8469:5169786] [bool test_div_broadcast()],[4 3 192 192 ],[SUCCEED]
2021-03-11 20:36:47.246816-0800 PyTorchPlayground[8469:5169786] [bool test_div_broadcast2()],[1 3 192 192 ],[SUCCEED]
```

Reviewed By: SS-JIA

Differential Revision: D27003417

fbshipit-source-id: 290f7e524eef4c444f8884fc1315151752e5ac31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants