-
Notifications
You must be signed in to change notification settings - Fork 26.3k
Add aten mkldnn add operator #19207
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
Add aten mkldnn add operator #19207
Conversation
Differential Revision: D14889477 Differential Version: 79209394
Differential Revision: D14889477 Differential Version: 79296740
Differential Revision: D14889477 Differential Version: 79299279
Differential Revision: D14889477 Differential Version: 79689941
Differential Revision: D14889477 Differential Version: 79698403
Differential Revision: D14889477 Differential Version: 79729544
Differential Revision: D14889477 Differential Version: 80386614
gchanan
left a comment
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.
approved once comments are addressed.
| ideep::tensor& y = itensor_from_mkldnn(other); | ||
|
|
||
| ideep::tensor& z = itensor_from_mkldnn(result); | ||
| const std::vector<float> scales{1.0, alpha.to<float>()}; |
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.
do we ever worry about mkldnn supporting double? We could technically write this generically with a dispatch macro.
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.
ideep::sum::compute only takes scales as float vector
Differential Revision: D14889477 Differential Version: 80533697
Differential Revision: D14889477 Differential Version: 80534300
Differential Revision: D14889477 Differential Version: 80534911
Differential Revision: D14889477 Differential Version: 80541661
Differential Revision: D14889477 Differential Version: 80564052
Differential Revision: D14889477 Differential Version: 80580147
Differential Revision: D14889477 Differential Version: 80683615
Differential Revision: D14889477 Differential Version: 80773728
Differential Revision: D14889477 Differential Version: 80780050
Differential Revision: D14889477 Differential Version: 80785116
Differential Revision: D14889477 Differential Version: 80799725
Summary: Pull Request resolved: pytorch/pytorch#19207 Reviewed By: dzhulgakov Differential Revision: D14889477 fbshipit-source-id: 2c5e5ea5dfc26a9c9a172c5fa2c6d7584b167e16
|
This pull request has been merged in 61d5a8d. |
Summary: Pull Request resolved: pytorch#19207 Reviewed By: dzhulgakov Differential Revision: D14889477 fbshipit-source-id: 2c5e5ea5dfc26a9c9a172c5fa2c6d7584b167e16
Stack:
:white_circle: #19633 Add is_mkldnn to at::Tensor 💚
:white_circle: #19204 Add aten mkldnn conv2d operator 💚
:white_circle: #19205 Add aten mkldnn ops: relu, max_pool2d and avg_pool2d 💚
:white_circle: #19206 Add aten mkldnn batch_norm operator 💚
:black_circle: #19207 Add aten mkldnn add operator 💚
:white_circle: #19209 Add aten mkldnn view operator 💚
:white_circle: #19210 Add aten mkldnn linear operator 💚
:white_circle: #19648 Adjust resnext run script 💚
Pull Request resolved: #19207
Differential Revision: D14889477