Skip to content

Conversation

@bddppq
Copy link
Contributor

@bddppq bddppq commented Jun 7, 2019

No description provided.

@pytorchbot pytorchbot added module: mkldnn Related to Intel IDEEP or oneDNN (a.k.a. mkldnn) integration module: operators labels Jun 7, 2019
namespace native {

namespace {
// TODO: move this to ideep
Copy link
Contributor Author

Choose a reason for hiding this comment

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

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.

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

Copy link
Contributor

@gchanan gchanan left a comment

Choose a reason for hiding this comment

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

would be good to have an MKLDNN person look at those details.

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.

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

@bddppq
Copy link
Contributor Author

bddppq commented Jun 7, 2019

@Jianhui-Li @XiaobingSuper @gujinghui Mind taking a look at the ideep_softmax_forward? ideally this will be moved to ideep itself.

@bddppq bddppq deleted the mkldnn-softmax branch June 7, 2019 22:36
zdevito pushed a commit to zdevito/ATen that referenced this pull request Jun 7, 2019
Summary: Pull Request resolved: pytorch/pytorch#21516

Differential Revision: D15712759

Pulled By: bddppq

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

@bddppq merged this pull request in 5744fb3.

@gujinghui
Copy link
Collaborator

@bddppq
Sure. Moved to ideep in PR #20569. Pls help review.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants