Skip to content

Conversation

@XiaobingSuper
Copy link
Collaborator

This PR is to solve the bug for clone a MKLDNN tensor, please see the issue #20895.

@pytorchbot pytorchbot added module: mkldnn Related to Intel IDEEP or oneDNN (a.k.a. mkldnn) integration module: operators labels May 25, 2019
Copy link
Contributor

@mrshenli mrshenli left a comment

Choose a reason for hiding this comment

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

@XiaobingSuper Thanks! Shall add your repro code in #20895 as a test for the mkldnn clone?

@XiaobingSuper
Copy link
Collaborator Author

@XiaobingSuper Thanks! Shall add your repro code in #20895 as a test for the mkldnn clone?

@mrshenli , added test case, thanks!

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@mrshenli has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Copy link
Contributor

@mrshenli mrshenli left a comment

Choose a reason for hiding this comment

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

Thanks for contributing!

zdevito pushed a commit to zdevito/ATen that referenced this pull request May 27, 2019
Summary:
This PR is to solve the bug for clone a MKLDNN tensor, please see the issue pytorch/pytorch#20895.
Pull Request resolved: pytorch/pytorch#20943

Differential Revision: D15511516

Pulled By: mrshenli

fbshipit-source-id: 05b41d6c7eaf8703521f4c768b8f26ec8501dc5e
@facebook-github-bot
Copy link
Contributor

@mrshenli merged this pull request in ebc8d71.

@XiaobingSuper XiaobingSuper deleted the mkldnn_clone branch May 28, 2019 00:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merged module: mkldnn Related to Intel IDEEP or oneDNN (a.k.a. mkldnn) integration open source

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants